Skip to content

Fix Flask health endpoint collision#278

Merged
Pigbibi merged 1 commit into
mainfrom
codex/fix-health-route-collision-20260701
Jul 1, 2026
Merged

Fix Flask health endpoint collision#278
Pigbibi merged 1 commit into
mainfrom
codex/fix-health-route-collision-20260701

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • avoid registering duplicate Flask endpoint names for health routes
  • keep /health and /healthz available on platforms with custom health checks
  • rename or isolate endpoint functions so they do not collide with the QPK health shim

Root cause

Cloud Run workers failed at import time with AssertionError: View function mapping is overwriting an existing endpoint function: health after register_health_endpoint(app) and local route functions both used endpoint name health.

Validation

  • python -m py_compile main.py
  • uvx ruff check main.py
  • static route-collision assertions

@Pigbibi Pigbibi merged commit 051312b into main Jul 1, 2026
2 checks passed
@Pigbibi Pigbibi deleted the codex/fix-health-route-collision-20260701 branch July 1, 2026 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant