Skip to content

feat: add HTTP streamable option to MCP client #16406

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 1 commit into
base: master
Choose a base branch
from

Conversation

victorazesc
Copy link

Summary

This PR introduces support for HTTP streamable responses when using the MCP client in the HTTP Request node.
This enables scenarios where the response body is large or long-lived (e.g., streamed AI completions).

How to test:

Added a new mcp client connector, added one http streamable source, like this image
image

Example workflow to test
image

Related Linear tickets, Github issues, and Community forum posts

No related tickets or issues at the moment.
Feature added proactively for advanced use cases with large or streamed data.

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

cubic reviewed 4 files and found no issues. Review PR in cubic.dev.

@n8n-assistant n8n-assistant bot added community Authored by a community member in linear Issue or PR has been created in Linear for internal review labels Jun 16, 2025
@Joffcom
Copy link
Member

Joffcom commented Jun 16, 2025

Hey @victorazesc,

Thanks for the PR, We have created "GHC-2586" as the internal reference to get this reviewed.

One of us will be in touch if there are any changes needed, in most cases this is normally within a couple of weeks but it depends on the current workload of the team.

@victorazesc
Copy link
Author

Ei@victorazesc,

Obrigado pelo PR. Criamos "GHC-2586" como referência interna para que isso seja revisado.

Um de nós entrará em contato caso seja necessária alguma alteração. Na maioria dos casos, isso normalmente ocorre em algumas semanas, mas depende da carga de trabalho atual da equipe.

Hey @victorazesc,

Thanks for the PR, We have created "GHC-2586" as the internal reference to get this reviewed.

One of us will be in touch if there are any changes needed, in most cases this is normally within a couple of weeks but it depends on the current workload of the team.

Thanks, happy to contribute! Let me know if I can help clarify anything during the review.

@Joffcom
Copy link
Member

Joffcom commented Jun 17, 2025

Hey @victorazesc,

Is this not the same as #15454?

@victorazesc
Copy link
Author

Hey @victorazesc,

Is this not the same as #15454?

Actually, this PR doesn’t work — the client connection is being used incorrectly.

@shawoozy-goed
Copy link

I need this! 👍

@victorazesc
Copy link
Author

I need this! 👍

I know, every body needs 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Authored by a community member in linear Issue or PR has been created in Linear for internal review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants