Skip to content

fix: use exec-form selfhost healthcheck#1421

Open
Frexxty wants to merge 1 commit into
UsefulSoftwareCo:mainfrom
Frexxty:fix/selfhost-healthcheck-no-shell
Open

fix: use exec-form selfhost healthcheck#1421
Frexxty wants to merge 1 commit into
UsefulSoftwareCo:mainfrom
Frexxty:fix/selfhost-healthcheck-no-shell

Conversation

@Frexxty

@Frexxty Frexxty commented Jul 16, 2026

Copy link
Copy Markdown

Summary

  • change the self-host Docker healthcheck to exec form so Docker does not invoke /bin/sh
  • keeps the existing Bun-based /api/health probe unchanged

Test Plan

  • docker build -f apps/host-selfhost/Dockerfile -t executor-selfhost:healthcheck-fix .
  • docker run -d --name selfhost-healthcheck-test -p 4789:4788 -e BETTER_AUTH_SECRET=... -e EXECUTOR_BOOTSTRAP_ADMIN_EMAIL=smoke@example.com -e EXECUTOR_BOOTSTRAP_ADMIN_PASSWORD=... -e EXECUTOR_WEB_BASE_URL=http://localhost:4789 executor-selfhost:healthcheck-fix
  • observed Docker health status become healthy
  • curl -fsS http://127.0.0.1:4789/api/health returned {"status":"ok"}

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