Issue 587/rest tool error (#778)

* added params extraction from url logic

Signed-off-by: Veeresh K <[email protected]>

* added params extraction from url logic

Signed-off-by: Veeresh K <[email protected]>

* Rebase and lint / test

Signed-off-by: Mihai Criveti <[email protected]>

---------

Signed-off-by: Veeresh K <[email protected]>
Signed-off-by: Mihai Criveti <[email protected]>
Co-authored-by: Mihai Criveti <[email protected]>

Author: nmveeresh

mcpgateway/services/tool_service.py

@@ -22,6 +22,7 @@
 import re
 import time
 from typing import Any, AsyncGenerator, Dict, List, Optional
+from urllib.parse import parse_qs, urlparse
 import uuid
 
 # Third-Party
@@ -756,6 +757,15 @@ async def invoke_tool(self, db: Session, name: str, arguments: Dict[str, Any], r
                             else:
                                 raise ToolInvocationError(f"Required URL parameter '{param}' not found in arguments")
 
+                    # --- Extract query params from URL ---
+                    parsed = urlparse(final_url)
+                    final_url = f"{parsed.scheme}://{parsed.netloc}{parsed.path}"
+
+                    query_params = {k: v[0] for k, v in parse_qs(parsed.query).items()}
+
+                    # Merge leftover payload + query params
+                    payload.update(query_params)
+
                     # Use the tool's request_type rather than defaulting to POST.
                     method = tool.request_type.upper()
                     if method == "GET":