MCP mTLS transport may inject ADC Authorization when an existing auth header is lowercase

🔴 Required Information

Describe the Bug:

In the MCP mTLS transport path, ADK may inject ADC / execution-environment Service Account credentials even when an Authorization header is already present on the MCP request.

The immediate cause is that _RefreshableAsyncCredentials.before_request() checks for an existing Authorization header case-sensitively:

if 'Authorization' in headers:

However, in the mTLS path, the MCP SDK uses httpx.AsyncClient, and ADK's _GoogleAuthAsyncTransport.handle_async_request() builds a plain dict from httpx.Request.headers before passing it to AsyncAuthorizedSession.request():

headers_dict = dict(request.headers)

httpx lowercases header names internally, so an existing Authorization header may be passed as authorization. In that case, the case-sensitive check misses the existing header and ADC credentials may be refreshed and injected into the same request.

As a result, an MCP server can receive an opaque access token for the runtime Service Account instead of the intended user / 3LO token.

Steps to Reproduce:

Use ADK v2.3.0 or current main.
Configure McpToolset with StreamableHTTPConnectionParams or SseConnectionParams.
Ensure the MCP request already has a user / 3LO Authorization header, for example through MCP auth credential handling or header_provider.
Run an MCP tool call in an environment where the mTLS transport path is enabled:
- GOOGLE_API_USE_CLIENT_CERTIFICATE is unset or set to true
- google.auth.aio is available
- ADC is available
- ADK can create _GoogleAuthAsyncTransport in _get_mtls_transport()
In _GoogleAuthAsyncTransport.handle_async_request(), dict(request.headers) can produce a lowercase authorization key.
_RefreshableAsyncCredentials.before_request() checks 'Authorization' in headers and misses the lowercase key.

Expected Behavior:

HTTP header names are case-insensitive. _RefreshableAsyncCredentials.before_request() should treat Authorization, authorization, and any other casing as the same header.

If any Authorization header is already present, ADK should not inject ADC / Service Account credentials.

Observed Behavior:

An existing Authorization header can be missed when its key is lowercase. In that case, ADK may refresh ADC credentials and inject an execution-environment token into the MCP request.

This can cause the MCP server to receive a Service Account / ADC token instead of the intended user / 3LO token.

Environment Details:

ADK Library Version (pip show google-adk): v2.3.0 and current main
Desktop OS: Linux
Python Version (python -V): Python 3.11

Model Information:

Are you using LiteLLM: N/A
Which model is being used: N/A. This issue occurs in the MCP HTTP transport path before model behavior is relevant.

🟡 Optional Information

Regression:

This appears related to the mTLS transport work introduced in v2.3.0. The issue is not about list_tools / discovery auth propagation. It affects the MCP HTTP request path when the mTLS transport bridges httpx requests into google-auth-aio AsyncAuthorizedSession.

Logs:

N/A. The behavior can be demonstrated at the header-normalization level:

dict(httpx.Request(..., headers={"Authorization": "Bearer user_token"}).headers)

contains authorization, not Authorization.

Screenshots / Video:

N/A.

Additional Context:

The existing Authorization header is not only a user-supplied static header. ADK has MCP header propagation paths intended to pass user / 3LO credentials to the MCP server during MCP tool execution.

After a 3LO flow completes, ADK resolves the user access token as an AuthCredential. In the MCP integration, that credential is converted into an HTTP header by McpToolset / McpTool. For OAuth2 credentials, the implementation generates a header like this:

headers = {"Authorization": f"Bearer {credential.oauth2.access_token}"}

For tool execution, the flow is roughly:

McpTool._run_async_impl() extracts auth headers from the resolved credential.
If header_provider is configured, it also obtains dynamic headers from ReadonlyContext.
The auth headers and dynamic headers are merged.
The merged headers are passed to MCPSessionManager.create_session(headers=final_headers).
MCPSessionManager passes those headers to the MCP SDK's streamablehttp_client / sse_client.

The discovery path (McpToolset.get_tools() / list_tools) has a similar flow through McpToolset._execute_with_session(), where headers from header_provider and exchanged credentials are merged and passed to create_session(headers=...).

Therefore, after a successful 3LO flow, the MCP HTTP request is expected to already contain:

Authorization: Bearer <user_3lo_access_token>

That header is required by the MCP server to identify the user context. If the later mTLS transport / Google auth bridge injects ADC or Service Account credentials into the same request, the principal observed by the MCP server can change from the user to the runtime Service Account.

This issue is not about failing to obtain the user token, nor is it about the MCP header propagation path failing to set a header. The problem occurs after the user / 3LO Authorization header has already reached create_session(headers=...): in the mTLS transport path, dict(request.headers) can represent it as lowercase authorization, and _RefreshableAsyncCredentials.before_request() does a case-sensitive check for Authorization.

This bug report focuses especially on the MCP tool-call request path. Independently of list_tools / discovery behavior, the tool-call HTTP request path should preserve and respect an existing Authorization header case-insensitively.

Suggested fix:

if any(key.lower() == "authorization" for key in headers):
    logger.debug("Authorization header already present, not overwriting")
    return

However, the header handling itself should still be fixed because HTTP header names are case-insensitive.

Related issues / PRs:

mcp_toolset is incompatible with mTLS #5361: MCP mTLS support discussion / predecessor context
chore(release/candidate): release 2.3.0 #6150: v2.3.0 release including the mTLS transport work
feat(mcp): Add session state-based JWT token propagation for MCP tools #3675: MCP header propagation feature, related but not the same bug
fix(api_registry): only attach ADC credentials to Google API hosts #6146: ADC credentials being attached in another component, related security theme but different code path

Minimal Reproduction Code:

import httpx

request = httpx.Request(
    "POST",
    "https://example.com/mcp",
    headers={"Authorization": "Bearer user_token"},
)

print(dict(request.headers))
# contains "authorization", not "Authorization"

How often has this issue occurred?:

Always (100%) when the mTLS transport path is active and the existing Authorization header is represented as lowercase authorization.

fix(mcp): Preserve existing auth in mTLS transport

@h-tsuboi918

Hi @h-tsuboi918 , Thank you for reporting this issue and submitting the fix in PR #6201.

The issue happens because google-adk looks specifically for a capitalized "Authorization" header. Since many HTTP clients use lowercase "authorization", the library misses your existing token and incorrectly adds its own Google Application Default Credentials (ADC).
Workaround: Until PR #6201 is merged and released, you can avoid this issue by ensuring any user-supplied authentication headers are explicitly passed using the capitalized "Authorization" key to satisfy the current check:

from google.adk.tools.mcp_tool import McpToolset
from google.adk.tools.mcp_tool.mcp_session_manager import StreamableHTTPConnectionParams

mcp_toolset = McpToolset(
    connection_params=StreamableHTTPConnectionParams(
        url="https://example-mcp-server.com/mcp",
        # Use exact capitalization to bypass the ADC injection logic
        headers={"Authorization": "Bearer YOUR_TOKEN"},  
    )
)

@sanketpatil06

Hi @sanketpatil06 , Thanks for the suggestion. I tested the suggested workaround against current main using the same McpToolset configuration shown in the comment:

toolset = McpToolset(
    connection_params=StreamableHTTPConnectionParams(
        url=url,
        headers={"Authorization": "Bearer user_3lo_token"},
        timeout=1.0,
        sse_read_timeout=1.0,
    )
)

await toolset.get_tools()

To make the behavior observable without depending on real ADC or a real remote MCP server, I used:

a local MCP-like HTTP server that records the received Authorization header
a minimal local ADC credential object whose refresh() sets service_account_token
the real current-main ADK path through McpToolset.get_tools(), MCPSessionManager.create_session(), MCP SDK streamablehttp_client, httpx.AsyncClient, _GoogleAuthAsyncTransport, and the current _RefreshableAsyncCredentials.before_request()

Here is the minimal repro I used:

import asyncio
import threading
from http.server import BaseHTTPRequestHandler, ThreadingHTTPServer

from google.auth.aio.transport.sessions import AsyncAuthorizedSession
from google.auth.credentials import Credentials
from google.adk.tools.mcp_tool import McpToolset
from google.adk.tools.mcp_tool.mcp_session_manager import (
    StreamableHTTPConnectionParams,
    _GoogleAuthAsyncTransport,
    _RefreshableAsyncCredentials,
)


class LocalAdc(Credentials):
    """Minimal ADC-like credential used to make injection observable."""

    def __init__(self):
        super().__init__()
        self.token = None
        self.refresh_count = 0

    @property
    def expired(self):
        return self.token is None

    def refresh(self, request):
        self.refresh_count += 1
        self.token = "service_account_token"


class McpLikeServer(BaseHTTPRequestHandler):
    """Local MCP-like endpoint that records the Authorization it receives."""

    seen_authorization = None

    def do_POST(self):
        self.__class__.seen_authorization = self.headers.get("Authorization")
        self.send_response(401)
        self.end_headers()

    def log_message(self, fmt, *args):
        pass


async def main():
    server = ThreadingHTTPServer(("127.0.0.1", 0), McpLikeServer)
    thread = threading.Thread(target=server.serve_forever, daemon=True)
    thread.start()

    host, port = server.server_address
    url = f"http://{host}:{port}/mcp"

    # Use the real current-main _RefreshableAsyncCredentials implementation.
    # LocalAdc avoids depending on a real ADC account while still showing
    # whether the ADC refresh/injection path is reached.
    adc = LocalAdc()
    auth_session = AsyncAuthorizedSession(
        _RefreshableAsyncCredentials(adc, target_host=f"{host}:{port}")
    )

    # Same McpToolset configuration as the suggested workaround:
    # explicitly use capitalized "Authorization".
    toolset = McpToolset(
        connection_params=StreamableHTTPConnectionParams(
            url=url,
            headers={"Authorization": "Bearer user_3lo_token"},
            timeout=1.0,
            sse_read_timeout=1.0,
        )
    )

    async def force_mtls_transport():
        return _GoogleAuthAsyncTransport(auth_session)

    # Force the same transport bridge used by the ADK mTLS path so the request
    # goes through McpToolset -> MCP SDK -> httpx -> _GoogleAuthAsyncTransport.
    toolset._mcp_session_manager._get_mtls_transport = force_mtls_transport

    try:
        await toolset.get_tools()
    except Exception:
        pass
    finally:
        await auth_session.close()
        server.shutdown()
        thread.join(timeout=5)

    print("configured Authorization: Bearer user_3lo_token")
    print(f"ADC refresh count: {adc.refresh_count}")
    print(f"server received Authorization: {McpLikeServer.seen_authorization}")


if __name__ == "__main__":
    asyncio.run(main())

Output on current main:

configured Authorization: Bearer user_3lo_token
ADC refresh count: 1
server received Authorization: Bearer service_account_token

So even with exact Authorization capitalization in StreamableHTTPConnectionParams.headers, the mTLS/httpx path still causes _RefreshableAsyncCredentials.before_request() to miss the existing user token and inject ADC credentials.

This workaround may be useful outside the mTLS/httpx path, but it does not appear to avoid the issue described here.

This is particularly problematic for 3LO flows. In that case, the user access token may have been obtained successfully and passed into the MCP request as Authorization: Bearer <user_token>, but the mTLS/httpx bridge can still cause ADK to inject the runtime ADC / Service Account token instead. From the MCP server's perspective, the request can then appear to come from the execution environment rather than the end user, which breaks user-scoped authorization and auditing.

fix: in mcp mtls logic, check for authorization header case insensiti…

Navigation Menu

Repository navigation

MCP mTLS transport may inject ADC Authorization when an existing auth header is lowercase #6200

🔴 Required Information

🟡 Optional Information

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

MCP mTLS transport may inject ADC Authorization when an existing auth header is lowercase #6200

Description

🔴 Required Information

🟡 Optional Information

Activity

sanketpatil06 commented on Jun 24, 2026

h-tsuboi918 commented on Jun 24, 2026

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions