/logs.json and /health URLs do not work if log prefix is /
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 72/100
Direzione di ricerca
Inizia leggendo la configurazione del gestore HTTP modificata nel commit 71b4f80 e segui come il gestore per log del prefisso radice interagisce con le route /logs.json e /health. Riproduci gli errori 404 con un prefisso di log /, poi verifica che entrambi gli URL rispondano correttamente mentre le route per log continuano a funzionare.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
We are getting HTTP 404 Not Found responses for /logs.json and /health URLs on our monitoring servers with Skylight v0.10.1.
Our log prefix is / and it seems that go http ServerMux prefers per-log HandleFunc with specified host and / path over global /logs.json and /health HandleFuncs. If logs have non-root prefix path, there is no conflict between per-log and global HandleFuncs, but there is if log prefix path is /.
I think this regression is introduced in v0.9.0 with commit 71b4f80 (cmd/skylight: implement c2sp.org/tlog-mirror read path) where ServerMux handlers got reworked. Before that HandleFunc patterns were more specific and not conflicted with /logs.json and /health, but with a per-log HandleFunc multiplexer the common per-log path became / in our case and that became more specific that /logs.json and /health.
- Lingua principale
- Go
- Stelle
- 318
- Fork
- 32
- Merge medio
- 5m
- PR unite (30g)
- 1
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di FiloSottile/sunlight
-
Migration toolAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 45/100
FiloSottile/sunlight#72 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 42/100
FiloSottile/sunlight#71 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
FiloSottile/sunlight#44 · 1 commento ·
-
Prefer publishing full tilesAperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
FiloSottile/sunlight#33 · 4 commenti ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 30/100
FiloSottile/sunlight#32 · 2 commenti ·
Tutte le issue di FiloSottile/sunlight
Issue simili
-
Cluster details: the Control Plane version field has a different border color than its icon boxApertakind/design sig/ui
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
kubermatic/dashboard#8449 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
filecoin-project/curio#1632 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 72/100
-
compose: derived node options overwrite each other's targetsForse già presa @yangjj-iso l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
I maintainer di solito rispondono entro 1 giorno