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

HTTP 307 from alertmanager is not respected #1368

Closed
lae opened this Issue Feb 4, 2016 · 3 comments

Comments

Projects
None yet
4 participants
@lae
Copy link

lae commented Feb 4, 2016

This occurs with alertmanager 1.0.0.beta1 and 2, and with alertmanager.url set to a url with a trailing slash (e.g. /usr/bin/prometheus -alertmanager.url=http://localhost:9093/) in Prometheus:

T 127.0.0.1:34951 -> 127.0.1.1:9093 [AP]
POST //api/v1/alerts HTTP/1.1.
Host: <host>:9093.
.
<payload>

T 127.0.1.1:9093 -> 127.0.0.1:34951 [AP]
HTTP/1.1 307 Temporary Redirect.
Location: /api/v1/alerts.
Date: Thu, 04 Feb 2016 01:03:39 GMT.

Prometheus doesn't attempt to POST the data again, so Alertmanager never sees the alerts. Prometheus should respect that response code and POST the data again (for this particular problem stripping out trailing slashes from alertmanager.url would be fine, but that's ugly).

@grobie grobie added the bug label Feb 4, 2016

@brian-brazil

This comment has been minimized.

Copy link
Member

brian-brazil commented Feb 4, 2016

Dupe of #668

@fabxc

This comment has been minimized.

Copy link
Member

fabxc commented Feb 4, 2016

That said, this particular case can be fixed by sanitizing the appending of the API endpoint #1370.

@lock

This comment has been minimized.

Copy link

lock bot commented Mar 24, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Mar 24, 2019

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.