fibers
fibers copied to clipboard
Support streaming responses in the responses in the fibers web server
Previously, any response would have to be entirely in memory prior to the first byte being sent to the client. This is fine for small response bodies like web pages, but can be undesirable when the response body is large or takes time to compute.
Can't say I I understand this well enough to properly review it, but I've enabled the CI and wrote some comments.
Looks like it fails for [email protected] because of (ice-9 exceptions) (apparently that's a supported Guile version?), can that be resolved?
Can't say I I understand this well enough to properly review it, but I've enabled the CI and wrote some comments.
Thanks :) I've pushed an updated patch now.
Looks like it fails for [email protected] because of (ice-9 exceptions) (apparently that's a supported Guile version?), can that be resolved?
It seems like I just left that import in the tests, I've removed it now so hopefully work with guile@2.
Unless maintainers chime in earlier, I intend to merge this after a month (guile-fibers isn't very active, hence a longer period than usual).
Unless maintainers chime in earlier, I intend to merge this after a month (guile-fibers isn't very active, hence a longer period than usual).
Thank you @emixa-d!