-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
[feat] Support application/event-stream #4
Comments
For the record: https://www.npmjs.com/package/fetch-event-stream may be helpful |
Hi, @onichandame. Yes, the SSE endpoints is not supported in I can add this feature to |
@vladkens when the response content type is text/event-stream |
Hi, @onichandame. I have looked into the question. Initially, the correct solution seems to be to use The library you gave as an example solves this problem. But, it do do manual HTTP parsing, which seems a bit more extensive and complicated than the basic functionality of the library. At this stage, if you don't have many methods, you can override specific method using |
@vladkens thank you! extending the ApiClient is exactly what I have been doing now. the DX is ok as SSE endpoints don't change often |
Now the SSE endpoints are treated as a short-lived request. Ideally such api should return an async iterator
The text was updated successfully, but these errors were encountered: