Skip to content

fix: MCP SSE cannot list tool #7420

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

chunlampang
Copy link

@chunlampang chunlampang commented Apr 3, 2025

An incorrect validation is used and let the SSE url always invalid.

a incorrect validation is used and make the SSE url always invalid
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. bug Something isn't working labels Apr 3, 2025
@github-actions github-actions bot added bug Something isn't working and removed bug Something isn't working labels Apr 3, 2025
@github-actions github-actions bot added bug Something isn't working and removed bug Something isn't working labels Apr 3, 2025
@arpit-patel1
Copy link

This is now resolved in new langflow >1.3.0

@ogabrielluiz
Copy link
Contributor

Hey @arpit-patel1

So it is working now, right?

@arpit-patel1
Copy link

Sorry, i seem commented on wrong issue. This is not the mcp issue i raised. Apologies

@chunlampang
Copy link
Author

chunlampang commented Apr 7, 2025

Maybe I mentioned on an incorrect issue. Let me repeat my issue.

Reproduction

  1. First follow this example to create a SSE mcp server
    https://github.com/modelcontextprotocol/typescript-sdk?tab=readme-ov-file#http-with-sse
  2. Then, input the sse url to the langflow MCP component.
  3. It will fail to query the tool list

If you direcly open the SSE url on browser, the url is accessible and sent the message url via event-stream, but the response will not be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants