Skip to content
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

Provide decorator for stream consumer through pipe #276

Closed
tzkmx opened this issue Jul 29, 2023 · 2 comments
Closed

Provide decorator for stream consumer through pipe #276

tzkmx opened this issue Jul 29, 2023 · 2 comments

Comments

@tzkmx
Copy link

tzkmx commented Jul 29, 2023

However I strongly feel that original API should be built on top of stream not the event

Originally posted by @amitguptagwl in #166 (comment)

Extending the Stream prototype has been tried but created issues that were reverted at the immediate posterior release.

There are several issues raised around support for Streams, and probably without breaking the current API, we can provide a decorator for request.pipe() that would make easier to integrate with handlers that expect a ReadableStream rather than json or urlencoded body.

@eugef
Copy link
Owner

eugef commented Aug 7, 2023

I think this is related to #277

@github-actions
Copy link

github-actions bot commented Oct 7, 2023

Stale issue message

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

No branches or pull requests

2 participants