Skip to content

fix(http): well-known mitm propagates original headers#406

Merged
Cali0707 merged 1 commit intocontainers:mainfrom
marcnuri-forks:fix/wellknown-mitm
Oct 24, 2025
Merged

fix(http): well-known mitm propagates original headers#406
Cali0707 merged 1 commit intocontainers:mainfrom
marcnuri-forks:fix/wellknown-mitm

Conversation

@manusa
Copy link
Member

@manusa manusa commented Oct 24, 2025

Token authorization in MCP Inspector now works fine in standard browsers with CORS enabled (no need to disable CORS).

localhost_6274__MCP_PROXY_AUTH_TOKEN=978b1c0082421360977d67b9d1e7ec7918e6b2c5b80c9813eb0306c7e61f87f5 (1)

Signed-off-by: Marc Nuri <marc@marcnuri.com>
@manusa manusa requested review from Cali0707 and matzew October 24, 2025 11:14
Copy link
Collaborator

@Cali0707 Cali0707 left a comment

Choose a reason for hiding this comment

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

This works for me as well - thanks for fixing the header propagation @manusa !

LGTM

@Cali0707 Cali0707 merged commit e86686a into containers:main Oct 24, 2025
6 checks passed
@manusa manusa deleted the fix/wellknown-mitm branch October 28, 2025 13:42
@manusa manusa added this to the 0.1.0 milestone Oct 28, 2025
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.

2 participants