Add safe url parsing and fix openapi converter comments

Author: h3xxit

plugins/communication_protocols/http/src/utcp_http/http_communication_protocol.py

@@ -396,7 +396,7 @@ def _build_url_with_path_params(self, url_template: str, tool_args: Dict[str, An
             if param_name in tool_args:
                 # Replace the parameter in the URL
                 # URL-encode the parameter value to prevent path injection
-                param_value = quote(str(tool_args[param_name]))
+                param_value = quote(str(tool_args[param_name]), safe="")
                 url = url.replace(f'{{{param_name}}}', param_value)
                 # Remove the parameter from arguments so it's not used as a query parameter
                 tool_args.pop(param_name)

plugins/communication_protocols/http/src/utcp_http/openapi_converter.py

@@ -65,14 +65,13 @@ def __init__(self, openapi_spec: Dict[str, Any], spec_url: Optional[str] = None,
             openapi_spec: Parsed OpenAPI specification as a dictionary.
             spec_url: Optional URL where the specification was retrieved from.
                 Used for base URL determination if servers are not specified.
-            call_template_name: Optional custom name for the call_template. If not
-                provided, derives name from the specification title.
+            call_template_name: Optional custom name for the call_template if 
+            the specification title is not provided.
         """
         self.spec = openapi_spec
         self.spec_url = spec_url
         # Single counter for all placeholder variables
         self.placeholder_counter = 0
-        # If call_template_name is None then get the first word in spec.info.title
         if call_template_name is None:
             call_template_name = "openapi_call_template_" + uuid.uuid4().hex
         title = openapi_spec.get("info", {}).get("title", call_template_name)

plugins/communication_protocols/http/src/utcp_http/sse_communication_protocol.py

@@ -329,7 +329,7 @@ def _build_url_with_path_params(self, url_template: str, tool_args: Dict[str, An
             if param_name in tool_args:
                 # Replace the parameter in the URL
                 # URL-encode the parameter value to prevent path injection
-                param_value = quote(str(tool_args[param_name]))
+                param_value = quote(str(tool_args[param_name]), safe="")
                 url = url.replace(f'{{{param_name}}}', param_value)
                 # Remove the parameter from arguments so it's not used as a query parameter
                 tool_args.pop(param_name)

plugins/communication_protocols/http/src/utcp_http/streamable_http_communication_protocol.py

@@ -354,7 +354,7 @@ def _build_url_with_path_params(self, url_template: str, tool_args: Dict[str, An
             if param_name in tool_args:
                 # Replace the parameter in the URL
                 # URL-encode the parameter value to prevent path injection
-                param_value = quote(str(tool_args[param_name]))
+                param_value = quote(str(tool_args[param_name]), safe="")
                 url = url.replace(f'{{{param_name}}}', param_value)
                 # Remove the parameter from arguments so it's not used as a query parameter
                 tool_args.pop(param_name)