Missing Health Check Endpoints in All Microservices - Team 065
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 42/100
Línea de trabajo
Empieza comparando README.md con los tres puntos de entrada: user_service/app.py, product_service/app.py y order_service/app.py. Ejecuta GET /health en cada servicio y define como completado que cada endpoint devuelva 200 con JSON que informe del estado del servicio y de la conectividad con la base de datos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Description
The architecture diagram in README.md documents /health endpoints for all three microservices (order_service, product_service, user_service), but these endpoints are not implemented in the actual code.
Current Behavior
- Calling
GET /healthon any service returns 404 Not Found - No health check endpoints exist in app.py files
Expected Behavior
Each service should respond to GET /health with:
- Status code: 200 OK
- JSON response indicating service health and database connectivity
Location
user_service/app.pyproduct_service/app.pyorder_service/app.py
Impact
- Cannot use health checks for container orchestration
- Documentation-code inconsistency
- Missing critical production feature
Suggested Solution
Implement /health endpoint in each service that:
- Returns 200 OK if service is healthy
- Checks database connectivity
- Returns appropriate JSON response
- Lenguaje dominante
- Python
- Estrellas
- 3
- Forks
- 8
- Merge medio
- 1 h 6 min
- PR fusionados (30 d)
- 3
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de keploy/ecommerce_sample_app
-
Improve and restructure README documentation for better clarity Issue 161Posiblemente ocupada @Jerry-145 la tomó hace 247 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
-
Hardcoded Credentials Security Issue T152Quizá libre de nuevo @Talaviya-Sarthak la tomó hace 247 días y no hay ningún pull request abierto. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 32/100
keploy/ecommerce_sample_app#7 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
keploy/ecommerce_sample_app#6 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 55/100
keploy/ecommerce_sample_app#5 · 1 comentario ·
Todos los issues de keploy/ecommerce_sample_app
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
-
[oblt-aw][autodoc] Document obs-aw-automerge contract and fix automerge doc inconsistenciesPosiblemente ocupada @github-actions la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
admission-deferred priority: medium status: ready work-class: scoped
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
melodic-software/claude-code-plugins#10887 · 4 comentarios ·
Los mantenedores suelen responder en 1 día
-
backend bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
PoCInnovation/Distribox#255 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
google/langextract#598 ·
Los mantenedores suelen responder en 1 día