Skip to content

Recognize bearer auth tokens for streamable http#43

Merged
aniket-s-kulkarni merged 2 commits into
mainfrom
remote-support-step-1
Jul 25, 2025
Merged

Recognize bearer auth tokens for streamable http#43
aniket-s-kulkarni merged 2 commits into
mainfrom
remote-support-step-1

Conversation

@aniket-s-kulkarni
Copy link
Copy Markdown
Contributor

  • Adds basic framework for streamable http
  • Adds framework to support bearer auth tokens, if specified, when in streamable http mode.

@aniket-s-kulkarni aniket-s-kulkarni merged commit 6822e28 into main Jul 25, 2025
2 checks passed
@rdkworld
Copy link
Copy Markdown

Thanks @aniket-s-kulkarni for making this available. With this PR being merged,

  1. I assume we can connect to MCP server endpoint http://127.0.0.1:8000/mcp with streamable http and should I provide any Bearer token for my local testing.
  2. Our Dremio (software) co-ordinator is used very lightly, Do you think it is ok to deploy on same co-ordinator (assuming port 8000 is not used for anything Dremio) or recommend deploying on a different server

@aniket-s-kulkarni
Copy link
Copy Markdown
Contributor Author

aniket-s-kulkarni commented Jul 25, 2025

Thanks @aniket-s-kulkarni for making this available. With this PR being merged,

  1. I assume we can connect to MCP server endpoint http://127.0.0.1:8000/mcp with streamable http and should I provide any Bearer token for my local testing.

  2. Our Dremio (software) co-ordinator is used very lightly, Do you think it is ok to deploy on same co-ordinator (assuming port 8000 is not used for anything Dremio) or recommend deploying on a different server

@rdkworld this is a preparatory PR. There will be a few more before the functionality is complete and can be used.

@rdkworld
Copy link
Copy Markdown

Oh ok, Thanks for the clarification @aniket-s-kulkarni , will wait for complete functionality. I guess I will hold off on wrapper/proxy I was building over stdio server since that would be throw-away effort once streamable-http support is available

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants