cube-js/cube

Need base Path for health-checks

Ouverte

#4 808 ouverte le 27 juin 2022

Voir sur GitHubGuide contributeur
 (2 commentaires) (1 réaction) (0 personne assignée)Rust (1 965 forks)batch import
help wantedloggedreviewed

Métriques du dépôt

Stars
 (19 563 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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

Guide contributeur

Direction de recherche
Étudiez comment Cube.js gère basePath pour d'autres points de terminaison et appliquez le même modèle aux points de terminaison de vérification de santé /livez et /readyz. Examinez la configuration du serveur et le middleware pour garantir que les routes de vérification de santé respectent basePath.
Stack technique
nodejstypescript
Domaine
backendapidevops
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitNode.js
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70