Update and clarify Security Model page
@jan--f is already working on this.
Since Oct 24, 2024.
- Dominant language
- TypeScript
- Stars
- 717
- Forks
- 1.2k
- Avg merge
- 3d 15h
- Merged PRs (30d)
- 19
Description
https://prometheus.io/docs/operating/security/#security-model is somewhat out of date WRT TLS and basic auth support.
I also think that the assumptions about how to handle /metrics endpoints should be clarified. It is by design that metrics are exposed, and that needs to pay attention about who has access to those endpoints and what information and DoS potential is exposed. Issues like https://github.com/prometheus/prometheus/issues/11290 create the impression that this exposure is surprising or a security flaw. See also https://kccnceu2022.sched.com/event/ytmB/how-attackers-use-exposed-prometheus-server-to-exploit-kubernetes-clusters-david-de-torres-huerta-miguel-hernandez-sysdig , which is less naive than the report above but also creates the impression that the design is flawed and that a "secure by default" setup is the expectation.
The Security Model page should not only make it very clear, even for a naive reader, that the /metrics endpoint of monitored targets and the query endpoints for the Prometheus server must not be accessible to the public internet in general, but it should also explain why this setup is a deliberate decision and a "secure by default" setup wouldn't really work.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.