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

Support TLS for reporting internal spans from Jaeger query to collector #2339

Open
kvaidas opened this issue Jul 10, 2020 · 3 comments
Open

Comments

@kvaidas
Copy link
Contributor

kvaidas commented Jul 10, 2020

Requirement - what kind of business use case are you trying to solve?

TLS configuration options in jaeger-query - for reporting its own spans to the collector

Problem - what in Jaeger blocks you from solving the requirement?

TLS configuration options like tls.server-name or tls.skip-host-verify are not available in jaeger-query

Proposal - what do you suggest to solve the problem or improve the existing situation?

I believe the libraries used in jaeger might already have this functionality but it needs to be exposed through cli parameters and/or envvars

@pavolloffay
Copy link
Member

Jaeger query uses https://github.com/jaegertracing/jaeger-client-go to report spans to the collector. The go client is configurable via env vars (see https://www.jaegertracing.io/docs/1.18/client-features/). At the moment there is no TLS support in the client libraries.

@pavolloffay pavolloffay changed the title TLS parameters in jaeger-query (for connecting to jaeger-collector) Support TLS for reporting internal spans from Jaeger query to collector Jul 10, 2020
@kvaidas
Copy link
Contributor Author

kvaidas commented Jul 10, 2020

As I understand #2337 should create some groundwork for this to be implemented.

@pavolloffay
Copy link
Member

no #2337 introduces TLS to query endpoints.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants