Skip to content

fix: use exec-form selfhost healthcheck - #1421

Merged
RhysSullivan merged 1 commit into
UsefulSoftwareCo:mainfrom
Frexxty:fix/selfhost-healthcheck-no-shell
Aug 3, 2026
Merged

fix: use exec-form selfhost healthcheck#1421
RhysSullivan merged 1 commit into
UsefulSoftwareCo:mainfrom
Frexxty:fix/selfhost-healthcheck-no-shell

Conversation

@Frexxty

@Frexxty Frexxty commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

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"}

@RhysSullivan
RhysSullivan merged commit 45d020c into UsefulSoftwareCo:main Aug 3, 2026
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