From 16a6adfda9131a3fada59d3745490a412fd9b5a3 Mon Sep 17 00:00:00 2001 From: Anso Date: Fri, 10 Apr 2026 22:16:01 -0400 Subject: [PATCH] docs(dockerfile): fix stale entrypoint comment (#504) The comment next to the ENTRYPOINT directive still described the old default ("fixes volume ownership as root then drops to sencho via su-exec"), contradicting the updated explanatory block a few lines up. Reword to match the new default: exec as root, drop to $SENCHO_USER only when opted in. --- Dockerfile | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Dockerfile b/Dockerfile index 3fa4c8fe..169e09a1 100644 --- a/Dockerfile +++ b/Dockerfile @@ -186,7 +186,8 @@ EXPOSE 3000 HEALTHCHECK --interval=30s --timeout=5s --start-period=15s --retries=3 \ CMD node -e "const h=require('http');h.get('http://localhost:3000/api/health',r=>{process.exit(r.statusCode===200?0:1)}).on('error',()=>process.exit(1))" -# Entrypoint fixes volume ownership as root then drops to sencho via su-exec. +# Entrypoint ensures /app/data is writable and execs the CMD as root by default, +# or drops to $SENCHO_USER via su-exec when that env var is set (see comment above). # CMD provides the default arguments passed through to the entrypoint. ENTRYPOINT ["/usr/local/bin/docker-entrypoint.sh"] CMD ["node", "dist/index.js"]