-
Notifications
You must be signed in to change notification settings - Fork 2.5k
[mcp-adapters] Request headers are still not being set for Legacy SSE transport #8194
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
Comments
@gauravmindzk I think this may have been fixed in v0.4.3, but can you please test with I'll leave this open for now, but please report back what you find and close if things are working for you. Thanks! |
Migrating issues over to |
hi @benjamincburns , will try again and let you know, thanks for the fix |
Not sure if relevant, but I am doing a similar thing, and for example, with code like
Still seeing that the Authorization header is missing in the request with v0.4.5 |
@botkevin what happens if you drop the |
@benjamincburns Still doesn't work. I first get |
Pretty sure this is the same issue as langchain-ai/langchainjs-mcp-adapters#65 (comment), I believe that SSE header support was just never supported in lieu of Streamable HTTP |
BREAKING_CHANGE: Drops the `useNodeEventSource` flag and stops passing an empty eventSourceInit object to Streamable HTTP and SSE transport constructors. If there continues to be header passing issues as a result of the way `SSEClientTransport` and `StreamableHTTPClientTransport` handle EventSource initialization, the correct fix will be to PR a change to the MCP SDK that changes how headers are populated on EventSource initialization. fixes #8196 fixes #8194
Uh oh!
There was an error while loading. Please reload this page.
my app’s backend is written in nodejs and I want to develop a chatbot in my application powered by Pinecone Assistant’s MCP server.
I’ve found the javascript code for uploading data to the Pinecone Assistant ( Upload files - Pinecone Docs )
I want a similar code for ( Use an Assistant MCP server - Pinecone Docs ) this .
Below is my agent’s code using Python:
I want its equivalent in javascript .
I came across Langchain’s github repo GitHub - langchain-ai/langchainjs-mcp-adapters: Adapters for integrating Model Context Protocol (MCP) tools with LangChain.js applications, supporting both stdio and SSE transports.
and tried setting up the code .
I'm getting an error when running this agent.
Package Versions:
The text was updated successfully, but these errors were encountered: