cube-js/cube

Need base Path for health-checks

Open

#4.808 geöffnet am 27. Juni 2022

Auf GitHub ansehen
 (2 Kommentare) (1 Reaktion) (0 zugewiesene Personen)Rust (1.965 Forks)batch import
help wantedloggedreviewed

Repository-Metriken

Stars
 (19.563 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 5T 16h) (138 gemergte PRs in 30 T)

Beschreibung

Is your feature request related to a problem? Please describe.

Hi I am currently deploying Cubejs behing Nginx Proxy so I should add prefix path to the Node. As the basePath configuration works very well for /v1/load, I could not hit health-checks related endpoint, since the Nginx controller will re-route the request incorrectly.

Describe the solution you'd like I was wondering, could we add /basePath prefix to livez and readyz ?. Is there any specific consideration why we did not set this up in the first place ?

Other Endpoint References

Requested Endpoints to change

Contributor Guide

Research-Richtung
Untersuche, wie Cube.js basePath für andere Endpunkte handhabt, und wende dasselbe Muster auf die Health Check Endpunkte /livez und /readyz an. Überprüfe die Serverkonfiguration und Middleware, um sicherzustellen, dass die Health Check Routen basePath respektieren.
Tech Stack
nodejstypescript
Domain
backendapidevops
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitNode.js
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70