jaeger icon indicating copy to clipboard operation
jaeger copied to clipboard

[Feature]: Support OTLP Authentication

Open rmarasigan opened this issue 2 years ago • 6 comments

Requirement

As a Jaeger user, I think adding the functionality of "auth" under the Jaeger Collector OTLP would be nice to have.

So when the OpenTelemetry Collector tries to communicate with the Jaeger Collector OTLP endpoint it will require it to export the trace data with authentication. It can be optional tho.

Here are some sample auth that is nice to have:

Problem

When the OpenTelemetry Collector is exporting the traces to the Jaeger Collector, we wanted somehow the OTLP Endpoint to have an authentication so there will be limited access to the said endpoint and in a more secure way. But right now, it just directly exports our data to the Jaeger Endpoint without any authentication which might cause some security issues.

Proposal

No response

Open questions

No response

rmarasigan avatar Jan 13 '23 06:01 rmarasigan

Hi @yurishkuro , is this issue still open, if yes may I proceed further?

psk001 avatar Jun 14 '23 07:06 psk001

Go ahead. My preference would be to not implement anything but check if OTEL Collector's receiver already supports auth. If it does not, I would not implement it in Jaeger either.

yurishkuro avatar Jun 14 '23 23:06 yurishkuro

Cool , I'll move as suggested

psk001 avatar Jun 15 '23 02:06 psk001

As mentioned in openetelemtry collector contrib, here , opentelemetry receiver supports http authentication using username and password. Also the discussion here and here have explanations.

psk001 avatar Jun 19 '23 07:06 psk001

What is the way forward for this @yurishkuro ?

psk001 avatar Jul 01 '23 23:07 psk001

@yurishkuro OTEL Collector's receiver Implemented HTTP Basic server auth. I think we are ready to go. Can I work on this?

varshith257 avatar Mar 16 '24 19:03 varshith257

Are there any plans to add oidc as well?

fcuello-fudo avatar Aug 14 '24 12:08 fcuello-fudo

Jaeger v2 uses OTLP receivers from OTEL collector, so we support the same features. We are not going to add new ones to jaeger v1.

yurishkuro avatar Aug 14 '24 14:08 yurishkuro