cube-js/cube

Need base Path for health-checks

Aperta

#4808 aperta il 27 giu 2022

Vedi su GitHubGuida contributor
 (2 commenti) (1 reazione) (0 assegnatari)Rust (1965 fork)batch import
help wantedloggedreviewed

Metriche repository

Star
 (19.563 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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

Guida contributor

Direzione di ricerca
Indaga come Cube.js gestisce basePath per altri endpoint e applica lo stesso pattern agli endpoint di health check /livez e /readyz. Esamina la configurazione del server e il middleware per garantire che le route di health check rispettino basePath.
Tech stack
nodejstypescript
Dominio
backendapidevops
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70