Skip to content

Conversation

@mishushakov
Copy link
Member

@mishushakov mishushakov commented Oct 31, 2025

Note

Adds websocket-aware healthcheck and sets a sandbox ready_cmd; exposes ContextWebSocket.is_connected().

  • Server (FastAPI):
    • Healthcheck: /health now returns 503 until any default context websocket is connected; 200 when ready.
  • WebSocket/Messaging:
    • Add ContextWebSocket.is_connected() to report connection status.
  • Config:
    • Set ready_cmd = "curl -sf http://localhost:49999/health" in template/e2b.toml.

Written by Cursor Bugbot for commit 98184c9. This will update automatically on new commits. Configure here.

@app.get("/health")
async def get_health():
return "OK"
websockets_ready = True
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are you getting some 503s?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jakubno never during start, because it calls connect on create_context

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am getting "curl: (52) Empty reply from server" instead, because health check endpoint is not up

cursor[bot]

This comment was marked as outdated.

@mishushakov mishushakov marked this pull request as draft November 3, 2025 14:12
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.

3 participants