-
Notifications
You must be signed in to change notification settings - Fork 4
[HTTP AUTH SPIKE - do not merge yet] Add HTTP transport with Auth0 middleware and configuration updates #61
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
How restricted is |
Good call, as of now, it looks like both /authorize and /callback handlers are vulnerable to open redirect attacks, but we are not planning to merge these changes to main. According to the MCP authorisation spec I was only expecting to implement the (meanwhile I'll convert this PR to draft as well, thanks for the security ping😂) |
Bearer realm=
toBearer resource_metadata=
[Note] We are not planning to merge the extra auth flow endpoints apart from the changes in the
handleProtectedResourceMetadata
. This PR is for the rest of the team to test the auth changes locally.