-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Allow alertmanager to be run behind a reverse proxy #212
Comments
FYI I just did https://github.com/novilabs/alertmanager/commit/50937b09100abcc1d28f933baa38e2b22aa0ef01 to be able to use a prefix path of |
@a86c6f7964 did you take a look on the pull request mentioned here? It seems to be a better solution. |
It never got fixed so far unfortunately. Pull requests very welcome, I think all core contributors are very overworked right now. |
Just looking at it, I got it almost working, except for one thing: the |
Without a closer look – that doesn't seem like something that should depend on passing through a template engine. |
@fabxc True. The alternative is to make all the paths between the assets relative. Trying that out now, it seems to work. It would mean that one has to be careful to not introduce any absolute paths anywhere, ever. But requiring a path prefix everywhere in the HTML+JS would have the same problem. Should I go for relative paths then? |
Sure, it's not that many files and won't change frequently. |
+1 to relative paths. |
Cool. Fix is out in https://github.com/prometheus/alertmanager/pull/289/files. |
This addresses the misleading error messages reported in #212 Explanation: #282 (comment) Fixes #282
I made it work like this ( |
The
-web.external-url
flag is no longer respected when loading assets. It's no longer possible to run alertmanager behind a reverse proxy.The text was updated successfully, but these errors were encountered: