Skip to content

[codex] add exec-server health check#21310

Open
richardopenai wants to merge 1 commit into
mainfrom
richard/exec-server-health-check
Open

[codex] add exec-server health check#21310
richardopenai wants to merge 1 commit into
mainfrom
richard/exec-server-health-check

Conversation

@richardopenai
Copy link
Copy Markdown
Contributor

Summary

Adds a lightweight GET /health probe on the exec-server websocket listener. The listener now performs a small preflight check before the websocket handshake, returns 200 OK with ok\n for the health route, and otherwise falls through to the existing websocket JSON-RPC path.

The preflight routing uses an explicit private enum so future probe routes such as /status can be added without turning the transport into a chain of boolean checks.

Validation

  • just fmt
  • cargo test -p codex-exec-server
  • just fix -p codex-exec-server

@richardopenai richardopenai marked this pull request as ready for review May 6, 2026 06:12
@starr-openai
Copy link
Copy Markdown
Contributor

lets make the HTTP service listen on a dedicated port, enabled via CLI like --status-endpoint-port. Check out the k8s docs for HTTP based health checks.

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.

2 participants