Skip to content

feat(mcp): Add AuthScheme and IdentityResolver support to HttpMcpProxy#1169

Merged
adwsingh merged 1 commit into
smithy-lang:mainfrom
DivyaMaddipudi:feat/mcp-authscheme-support
May 11, 2026
Merged

feat(mcp): Add AuthScheme and IdentityResolver support to HttpMcpProxy#1169
adwsingh merged 1 commit into
smithy-lang:mainfrom
DivyaMaddipudi:feat/mcp-authscheme-support

Conversation

@DivyaMaddipudi
Copy link
Copy Markdown
Contributor

Add proper auth separation to HttpMcpProxy alongside the existing signer() path. The new authScheme() + identityResolver() + signerContext() builder API follows the canonical smithy-java auth architecture (AuthScheme + IdentityResolver + Signer separation) used by ClientPipeline. This enables direct use of SigV4AuthScheme and SdkCredentialsResolver for AWS-authenticated MCP endpoints without needing a custom adapter class.

The existing signer() path remains unchanged for backward compatibility.

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Add proper auth separation to HttpMcpProxy alongside the existing
signer() path. The new authScheme() + identityResolver() +
signerContext() builder API follows the canonical smithy-java auth
architecture (AuthScheme + IdentityResolver + Signer separation) used
by ClientPipeline. This enables direct use of SigV4AuthScheme and
SdkCredentialsResolver for AWS-authenticated MCP endpoints without
needing a custom adapter class.

The existing signer() path remains unchanged for backward compatibility.
Copy link
Copy Markdown
Contributor

@franco2002lu franco2002lu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. cc: @adwsingh

@adwsingh adwsingh enabled auto-merge (rebase) May 11, 2026 19:15
@adwsingh adwsingh merged commit d488b60 into smithy-lang:main May 11, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants