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

grpc-web + envoy does not work #118

Open
ritchieanesco opened this issue Oct 16, 2024 · 0 comments
Open

grpc-web + envoy does not work #118

ritchieanesco opened this issue Oct 16, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@ritchieanesco
Copy link

Proposal

I've setup a client-side app with react which uses connectrpc's transport for grpc-web protocol and sends requests to envoy over HTTP/1.1 and gets proxied to wiremock server.

The wiremock stubs contains "grpc-status-name" : "OK" response header however it seems no headers get added when looking at envoy proxy logs. See excerpt from log:

2024-10-16 08:49:44 [2024-10-15 21:49:44.222][28][debug][http] [source/common/http/conn_manager_impl.cc:1516] [C0][S4239721515345536154] encoding headers via codec (end_stream=false):
2024-10-16 08:49:44 ':status', '200'
2024-10-16 08:49:44 'content-type', 'application/grpc-web+proto'
2024-10-16 08:49:44 'grpc-encoding', 'identity'
2024-10-16 08:49:44 'grpc-accept-encoding', 'gzip'
2024-10-16 08:49:44 'content-encoding', 'gzip'
2024-10-16 08:49:44 'x-envoy-upstream-service-time', '2139'
2024-10-16 08:49:44 'date', 'Tue, 15 Oct 2024 21:49:44 GMT'
2024-10-16 08:49:44 'server', 'envoy'

In addition to this, the response payload does not include the required grpc trailers as evidenced by this log:

2024-10-16 07:14:55 [2024-10-15 20:14:55.595][41][debug][http] [source/common/http/conn_manager_impl.cc:1533] [C0][S5164429598623932382] encoding trailers via codec:

2024-10-16 07:14:55 [2024-10-15 20:14:55.596][41][debug][http2] [source/common/http/http2/codec_impl.cc:1417] [C1] stream 1 closed: 0

The missing data from this response results in the react application throwing a missing trailers error.

wiremock-main.zip

Reproduction steps

Follow README to install and run the attached source code.

See Test mock server integration section to reproduce issue.

References

No response

@ritchieanesco ritchieanesco added the bug Something isn't working label Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant