Fix UrlRedactingFilter to keep arguments the type that they are + only filter out *mcp.gumloop.com* URLs

Author: rbehal

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "gumloop-mcp"
-version = "0.1.13"
+version = "0.1.14"
 description = "guMCP (Gumloop Model Context Protocol) SDK"
 readme = "README.md"
 requires-python = ">=3.10"

src/mcp/shared/logging_utils.py

@@ -9,46 +9,58 @@
 
 class UrlRedactingFilter(logging.Filter):
     """Redacts URLs in log messages to prevent credential leaks."""
-    
-    URL_PATTERN = re.compile(r'https?://[^\s]+')
-    
+
+    # Matches http/https, optional subdomains, mcp.gumloop.com, and the rest of the path
+    # (?:[a-zA-Z0-9-]+\.)* matches optional subdomains before mcp.gumloop.com
+    URL_PATTERN = re.compile(r"https?://(?:[a-zA-Z0-9-]+\.)*mcp\.gumloop\.com[^\s]*")
+
     @staticmethod
     def _redact_url(match):
         """Redact a URL to only show scheme and netloc."""
         url = match.group(0)
         parsed = urlparse(url)
         return f"{parsed.scheme}://{parsed.netloc}"
-    
+
     def filter(self, record: logging.LogRecord) -> bool:
         if isinstance(record.msg, str):
             record.msg = self.URL_PATTERN.sub(self._redact_url, record.msg)
-        
+
         if record.args:
             if isinstance(record.args, dict):
-                record.args = {
-                    k: self.URL_PATTERN.sub(self._redact_url, str(v))
-                    for k, v in record.args.items()
-                }
+                new_args = {}
+                for k, v in record.args.items():
+                    if isinstance(v, str):
+                        new_args[k] = self.URL_PATTERN.sub(self._redact_url, v)
+                    else:
+                        new_args[k] = v
+                record.args = new_args
             else:
                 record.args = tuple(
-                    self.URL_PATTERN.sub(self._redact_url, str(arg))
+                    (
+                        self.URL_PATTERN.sub(self._redact_url, arg)
+                        if isinstance(arg, str)
+                        else arg
+                    )
                     for arg in record.args
                 )
-        
+
         # Sanitize exception info if present
         if record.exc_info and record.exc_info[1]:
             exc = record.exc_info[1]
-            if hasattr(exc, 'args') and exc.args:
+            if hasattr(exc, "args") and exc.args:
                 redacted_args = tuple(
-                    self.URL_PATTERN.sub(self._redact_url, str(arg)) if isinstance(arg, str) else arg
+                    (
+                        self.URL_PATTERN.sub(self._redact_url, arg)
+                        if isinstance(arg, str)
+                        else arg
+                    )
                     for arg in exc.args
                 )
                 exc.args = redacted_args
-        
+
         return True
 
 
 def redact_url_logs(logger: logging.Logger) -> None:
     """Add URL redacting filter to logger to strip paths and credentials from URLs."""
     logger.addFilter(UrlRedactingFilter())
-