-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
bug: Compression plugin brotli is sending partial response #11079
Comments
|
Add the header is tricky part , but here is the fix for partial response. |
One can assign it to me. :) |
the transfer-encoding header maybe this reason why curl raise a error, could you try |
oh, HTTP2 is getting used, that's why Ref: https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Transfer-Encoding And looks like in HTTP2 it's not mandatory to send Ref: https://datatracker.ietf.org/doc/html/rfc7540#section-8.1.2.6. But the fix for the partial response is required. PR: #11087 solves that. |
Updated the issue and PR, please check once. @yuweizzz |
yes, the fix for the partial response is required, but now I am thinking if we need to clear somes headers from upstream when we use http2. so that we can avoid http2 protocol error. |
@yuweizzz don't you think the upstream should handle these protocol version nuances? |
Hi @yuweizzz, the body-related headers from the upstream are getting purged in
Body-related headers are getting cleared as it's a compression plugin, and will change/compress the response body. |
I had some test in local env, this part will handle by openresty correctly,
|
Current Behavior
Brotli plugin is sending partial response from the apisix.
Expected Behavior
The last chunk is not present in the response.
Full response should be provided.
eof
flag should not overwrite thearg[1]
, but it(arg[1]
) should be appended withcompressor:finish()
.Error Logs
Steps to Reproduce
Configure
brotli
, and enable the plugin, and try sending the request to the route.Environment
apisix version
): latestuname -a
): NAopenresty -V
ornginx -V
): NAcurl http://127.0.0.1:9090/v1/server_info
): NAluarocks --version
): NAThe text was updated successfully, but these errors were encountered: