Skip to content

Conversation

@manusa
Copy link
Member

@manusa manusa commented Aug 6, 2025

No description provided.

@manusa manusa added this to the 0.1.0 milestone Aug 6, 2025
@manusa manusa force-pushed the feat/oauth-authorization-server-endpoint branch from 9abb8df to 0da5b5b Compare August 6, 2025 11:43
@manusa manusa force-pushed the feat/oauth-authorization-server-endpoint branch from 0da5b5b to 3d23d94 Compare August 6, 2025 11:59
@manusa manusa merged commit aba5f54 into containers:main Aug 6, 2025
7 of 8 checks passed
@manusa manusa deleted the feat/oauth-authorization-server-endpoint branch August 6, 2025 12:51
mux.HandleFunc(healthEndpoint, func(w http.ResponseWriter, r *http.Request) {
w.WriteHeader(http.StatusOK)
})
mux.HandleFunc(oauthAuthorizationServerEndpoint, OAuthAuthorizationServerHandler(staticConfig))
Copy link
Member

Choose a reason for hiding this comment

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

@manusa this endpoint is served by oidc providers not mcp servers. Is there any reason for this?. If you implemented based on my suggestion on Slack, sorry it seems that I wasn't clear.

Copy link
Member

Choose a reason for hiding this comment

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

I think, you implemented this because mcp/inspector looks for this endpoint which is served by Keycloak in a different endpoint?.

Copy link
Member Author

Choose a reason for hiding this comment

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

Exactly, this is looked for by the MCP clients, I'm currently working on an implementation to proxy the following endpoints:

	oauthAuthorizationServerEndpoint = "/.well-known/oauth-authorization-server"
	oauthProtectedResourceEndpoint   = "/.well-known/oauth-protected-resource"
	openIDConfigurationEndpoint      = "/.well-known/openid-configuration"

This delegates everything to the auth server (Keycloak).
This seems to work with the inspector and very likely with VSCode.

I'll create a PR shortly

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