-
Notifications
You must be signed in to change notification settings - Fork 902
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
Security in Prometheus exporter #2400
Comments
Under which version is this feature released?... |
@sandeepsharat it is no released. This is a feature request |
Chiming in during triage - this seems very useful. Tagging @open-telemetry/wg-prometheus for their comments. |
This is a good idea, but might only apply to auto-instrumentation. From looking at manual instrumentation in a few languages, it looks like most allow you to either replace the HTTP handler, serve the endpoint using the prometheus client library, or require you to serve the http endpoint yourself. |
@dashpole do you think there's something we should add to the spec to support this, or should people open issues directly with various implementations to support it? thanks |
I would recommend people open issues directly with implementations, since most prometheus exporters don't directly configure the endpoint. I'm generally a in favor of using prometheus client libraries (which is recommended by the current spec) for serving, and think any additional capabilities around securing the endpoint probably belongs in prometheus clients, rather than in OTel. For things like auto-instrumentation, where we configure the endpoint, we should try and "pass through" configuration to prometheus client libraries as much as possible, rather than re-invent the wheel, and try to direct people to the prometheus clients if there are feature gaps. |
Thanks, makes sense, let's try this and people can loop back here as needed. |
What are you trying to achieve?
In the current spec for the Prometheus exporter https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/sdk_exporters/prometheus.md, it doesn't mention anything about securing the metrics endpoint.
When using the automatic instrumentation opentelemetry-javaagent.jar and the Prometheus exporter is enabled, the metrics endpoint (e.g. http://localhost:9464/) is not protected.
What did you expect to see?
There should be an option to use https and have the metrics endpoint protected by
Additional context.
Add any other context about the problem here. If you followed an existing documentation, please share the link to it.
The text was updated successfully, but these errors were encountered: