-
-
Notifications
You must be signed in to change notification settings - Fork 604
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
[feature] Proxy headers more options or proxy all #334
Comments
@telemmaite yeah, this is reasonable to have. We should not proxy all headers as those can contain a lot of headers that do not have much sense like Please look at #335 - is it ok? |
@FZambia looks great and thank you very much for your effort on this project! |
@telemmaite hi, released new version with new proxy features - v2.3.1 |
@FZambia thank you! Already using the |
Is your feature request related to a problem? Please describe.
We are using
distributed tracing
for example https://opentracing.io/, https://zipkin.io/ (https://zipkin.io/pages/architecture.html, https://github.com/openzipkin/b3-propagation)We have the option to trace request from from frontend > through backend > back to frontend.
This means a few
trace
headers are added, as quoted:Describe the solution you'd like
centrifugo/internal/proxy/http.go
Line 74 in 02d489a
Describe alternatives you've considered
Missing part of our tracing log for all proxied requests from centrifugo, if the headers are not passed our backend will assume it is the first entry point and we will add the headers on the trip back.
The text was updated successfully, but these errors were encountered: