fix: preserve iterable tools before validation and reject trailing slashes

Author: giulio-leone

src/openai/lib/_validation.py

@@ -10,7 +10,7 @@
 from typing import Any, Iterable, Optional
 
 _PROTOCOL_RE = re.compile(r"^https?://", re.IGNORECASE)
-_PATH_RE = re.compile(r"/.")
+_PATH_RE = re.compile(r"/.*$")
 
 
 def validate_network_policy_allowlist(

src/openai/resources/responses/responses.py

@@ -3534,10 +3534,13 @@ def _make_tools(tools: Iterable[ParseableToolParam] | Omit) -> List[ToolParam] |
     if not is_given(tools):
         return omit
 
-    validate_tools(tools)
+    # Materialise once so that validation doesn't consume a one-shot iterator
+    tools_list = list(tools) if not isinstance(tools, list) else tools
+
+    validate_tools(tools_list)
 
     converted_tools: List[ToolParam] = []
-    for tool in tools:
+    for tool in tools_list:
         if tool["type"] != "function":
             converted_tools.append(tool)
             continue