rootly-mcp-server 2.2.1__tar.gz → 2.2.3__tar.gz

{rootly_mcp_server-2.2.1 → rootly_mcp_server-2.2.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: rootly-mcp-server
-Version: 2.2.1
+Version: 2.2.3
 Summary: Secure Model Context Protocol server for Rootly APIs with AI SRE capabilities, comprehensive error handling, and input validation
 Project-URL: Homepage, https://github.com/Rootly-AI-Labs/Rootly-MCP-server
 Project-URL: Issues, https://github.com/Rootly-AI-Labs/Rootly-MCP-server/issues

{rootly_mcp_server-2.2.1 → rootly_mcp_server-2.2.3}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "rootly-mcp-server"
-version = "2.2.1"
+version = "2.2.3"
 description = "Secure Model Context Protocol server for Rootly APIs with AI SRE capabilities, comprehensive error handling, and input validation"
 readme = "README.md"
 requires-python = ">=3.10"

{rootly_mcp_server-2.2.1 → rootly_mcp_server-2.2.3}/src/rootly_mcp_server/__init__.py

@@ -14,10 +14,15 @@ Features:
 - Input validation and sensitive data masking
 """
 
+from importlib.metadata import PackageNotFoundError, version
+
 from .client import RootlyClient
 from .server import RootlyMCPServer
 
-__version__ = "2.1.0"
+try:
+    __version__ = version("rootly-mcp-server")
+except PackageNotFoundError:
+    __version__ = "dev"
 __all__ = [
     "RootlyMCPServer",
     "RootlyClient",

{rootly_mcp_server-2.2.1 → rootly_mcp_server-2.2.3}/src/rootly_mcp_server/server.py

@@ -386,18 +386,41 @@ class AuthenticatedHTTPXClient:
         if "params" in kwargs:
             kwargs["params"] = self._transform_params(kwargs["params"])
 
+        # Log incoming headers for debugging (before transformation)
+        incoming_headers = kwargs.get("headers", {})
+        if incoming_headers:
+            logger.debug(f"Incoming headers for {method} {url}: {list(incoming_headers.keys())}")
+
         # ALWAYS ensure Content-Type and Accept headers are set correctly for Rootly API
         # This is critical because:
-        # 1. FastMCP may pass headers from the MCP client request (e.g., Content-Type: application/json)
-        # 2. Per-request headers override httpx client defaults
-        # 3. We must force JSON-API content type regardless of what's passed in
+        # 1. FastMCP's get_http_headers() returns LOWERCASE header keys (e.g., "content-type")
+        # 2. We must remove any existing content-type/accept and set the correct JSON-API values
+        # 3. Handle both lowercase and mixed-case variants to be safe
         headers = dict(kwargs.get("headers") or {})
+        # Remove any existing content-type and accept headers (case-insensitive)
+        headers_to_remove = [k for k in headers if k.lower() in ("content-type", "accept")]
+        for key in headers_to_remove:
+            logger.debug(f"Removing header '{key}' with value '{headers[key]}'")
+            del headers[key]
+        # Set the correct JSON-API headers
         headers["Content-Type"] = "application/vnd.api+json"
         headers["Accept"] = "application/vnd.api+json"
         kwargs["headers"] = headers
 
-        # Call the underlying client's request method and let it handle everything
-        return await self.client.request(method, url, **kwargs)
+        # Log outgoing request
+        logger.debug(f"Request: {method} {url}")
+
+        response = await self.client.request(method, url, **kwargs)
+        logger.debug(f"Response: {method} {url} -> {response.status_code}")
+
+        # Log error responses (4xx/5xx)
+        if response.is_error:
+            logger.error(
+                f"HTTP {response.status_code} error for {method} {url}: "
+                f"{response.text[:500] if response.text else 'No response body'}"
+            )
+
+        return response
 
     async def get(self, url: str, **kwargs):
         """Proxy to request with GET method."""
@@ -543,6 +566,20 @@ def create_rootly_mcp_server(
 
         return endpoints
 
+    @mcp.tool()
+    def get_server_version() -> dict:
+        """Get the Rootly MCP server version.
+
+        Returns the current version of the deployed MCP server.
+        Useful for checking if the server has been updated.
+        """
+        from rootly_mcp_server import __version__
+
+        return {
+            "version": __version__,
+            "package": "rootly-mcp-server",
+        }
+
     async def make_authenticated_request(method: str, url: str, **kwargs):
         """Make an authenticated request, extracting token from MCP headers in hosted mode."""
         # In hosted mode, get token from MCP request headers

{rootly_mcp_server-2.2.1 → rootly_mcp_server-2.2.3}/tests/unit/test_http_headers.py

@@ -275,3 +275,44 @@ class TestFastMCPIntegrationScenario:
             call_kwargs = mock_httpx_client.request.call_args[1]
             assert call_kwargs["headers"]["Content-Type"] == "application/vnd.api+json"
             assert call_kwargs["headers"]["Accept"] == "application/vnd.api+json"
+
+    @pytest.mark.asyncio
+    async def test_lowercase_headers_removed_not_duplicated(self, mock_httpx_client):
+        """Test that lowercase headers from FastMCP are removed, not duplicated.
+
+        FastMCP's get_http_headers() returns lowercase keys like 'content-type'.
+        If we only SET 'Content-Type' without removing 'content-type', the dict
+        would have BOTH keys, causing the 415 error on the hosted server.
+
+        This is the ROOT CAUSE of the 415 error that only happens on hosted MCP:
+        - Local (stdio): No HTTP request context, get_http_headers() returns {}
+        - Hosted (SSE): HTTP request context exists, get_http_headers() returns lowercase headers
+        """
+        from rootly_mcp_server.server import AuthenticatedHTTPXClient
+
+        with patch.object(AuthenticatedHTTPXClient, "_get_api_token", return_value="test-token"):
+            client = AuthenticatedHTTPXClient()
+            client.client = mock_httpx_client
+
+            # FastMCP returns lowercase header keys from get_http_headers()
+            fastmcp_headers = {
+                "content-type": "application/json",  # lowercase from FastMCP!
+                "accept": "application/json",  # lowercase from FastMCP!
+                "authorization": "Bearer token",
+            }
+
+            await client.request("GET", "/v1/teams", headers=fastmcp_headers)
+
+            call_kwargs = mock_httpx_client.request.call_args[1]
+            headers = call_kwargs["headers"]
+
+            # Should have correct Content-Type (mixed case)
+            assert headers["Content-Type"] == "application/vnd.api+json"
+            assert headers["Accept"] == "application/vnd.api+json"
+
+            # Should NOT have lowercase duplicates - this is the key assertion!
+            assert "content-type" not in headers, "lowercase content-type should be removed"
+            assert "accept" not in headers, "lowercase accept should be removed"
+
+            # Other headers should be preserved
+            assert headers["authorization"] == "Bearer token"