Skip to content

Add hint to regarding CORS headers of Alertmanager - #3088

Merged
juliusv merged 1 commit into
prometheus:mainfrom
SoloJacobs:alphabet
Aug 27, 2026
Merged

Add hint to regarding CORS headers of Alertmanager#3088
juliusv merged 1 commit into
prometheus:mainfrom
SoloJacobs:alphabet

Conversation

@SoloJacobs

Copy link
Copy Markdown
Contributor

The current documentation specifies that only trusted users should have access to the http endpoints of Alertmanager. However, the browsers default Single Origin Policy pertains more to network access.

Thus, documenting the Access-Control-Allow-Origin: * headers is a good idea. The API Security section already hints at the configuration, but mentioning it in a Alertmanager specific location shouldn't hurt.

The current documentation specifies that only trusted users should have
access to the http endpoints of Alertmanager. However, the browsers
default Single Origin Policy pertains more to network access.

Thus, documenting the `Access-Control-Allow-Origin: *` headers is a good
idea. The `API Security` section already hints at the configuration,
but mentioning it in a Alertmanager specific location shouldn't hurt.

Signed-off-by: Solomon Jacobs <solomonjacobs@protonmail.com>

@juliusv juliusv left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I still think this is dangerous / unexpected default behavior for most users, but documenting it is a good start. The Prometheus server has the same CORS issue, except that it's read-only by default and doesn't allow people to write any data.

@juliusv
juliusv merged commit 9ece2ea into prometheus:main Aug 27, 2026
6 checks passed
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

Successfully merging this pull request may close these issues.

2 participants