Skip to content

Conversation

ihrpr
Copy link
Contributor

@ihrpr ihrpr commented May 12, 2025

We need to make sure Inspector also works with Streamable Http

@ihrpr ihrpr changed the title Simple Auth to use shttp in addition to sse Simple Auth Example to use shttp in addition to sse May 12, 2025
@ihrpr ihrpr marked this pull request as ready for review May 12, 2025 17:36
Copy link
Member

@pcarleton pcarleton left a comment

Choose a reason for hiding this comment

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

LGTM!

Thanks, tested this locally, and ran through the inspector auth flow

@ihrpr ihrpr merged commit fdb538b into main May 12, 2025
9 checks passed
@ihrpr ihrpr deleted the ihrpr/auth-shttp-example branch May 12, 2025 21:08
@lkchao-majority
Copy link

Hi @ihrpr
First of all, a big thank you for your work! Thanks to your example, I was able to create my own MCP server using a different OAuth provider.

I have a question: does the SDK actually support refresh tokens?
When I tried to manually trigger the refresh token workflow using the Inspector tool, I received a “Bad Request” error from my server.

This leads me to believe that the server might not be expecting the current request format on the same /token endpoint.

saqadri pushed a commit to saqadri/stdio-fixes that referenced this pull request Aug 6, 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.

3 participants