We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Versions of relevant software used v0.15.0
What happened The wrapHandler function does not check Trailers-Only case and adds extra data to trailers.
grpc-web/go/grpcweb/grpc_web_response.go
Lines 97 to 106 in 1d9bbb0
Instead of empty trailers in the case of Trailers-Only, I get a trailer with 0x80 0x0 0x0 0x0 0x0
What you expected to happen In the case of Trailers-Only, Trailer must remain empty. In the response from the server only headers are involved
How to reproduce it (as minimally and precisely as possible): Your library uses the traefik gateway, which allows the case of Trailers-Only.
https://github.com/traefik/traefik/blob/8174860770e536b4afb541e0ab13b3611a101430/pkg/middlewares/grpcweb/grpcweb.go#L15-L26
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Versions of relevant software used
v0.15.0
What happened
The wrapHandler function does not check Trailers-Only case and adds extra data to trailers.
grpc-web/go/grpcweb/grpc_web_response.go
Lines 97 to 106 in 1d9bbb0
It's about line 101
Instead of empty trailers in the case of Trailers-Only, I get a trailer with 0x80 0x0 0x0 0x0 0x0
What you expected to happen
In the case of Trailers-Only, Trailer must remain empty. In the response from the server only headers are involved
How to reproduce it (as minimally and precisely as possible):
Your library uses the traefik gateway, which allows the case of Trailers-Only.
https://github.com/traefik/traefik/blob/8174860770e536b4afb541e0ab13b3611a101430/pkg/middlewares/grpcweb/grpcweb.go#L15-L26
The text was updated successfully, but these errors were encountered: