Stu Mason
Stu Mason

Activity

Pull Request Opened

PR #70 opened: fix: batch fixes for #60, #38, #59, #67, #68

Summary

Five low-effort, high-impact fixes from open issues:

  • #68 — Suppress /up health check from nginx access logs (access_log off; log_not_found off)
  • #60 — Scheduler supervisor now uses schedule:work instead of a while/sleep/schedule:run loop
  • #38 — Fix ssh-keygen "Too many arguments" by using escapeshellarg() for all parameters
  • #59 — Clear git_commit_sha after deploying a specific commit so future deploys use HEAD
  • #67 — Container logging: generate docker/php-fpm.conf with catch_workers_output = yes, use errorlog channel (not stderr — which doesn't work with PHP-FPM), add logging guidance to coolify:install

Also closed #51 and #26 as out of scope.

Closes #60, closes #38, closes #59, closes #67, closes #68

Test plan

  • All 268 existing tests pass
  • PHPStan clean
  • Verify coolify:install shows logging recommendation
  • Verify generated Dockerfile includes php-fpm.conf COPY line
  • Verify coolify:provision sets LOG_STACK=daily,errorlog
+86
additions
-8
deletions
6
files changed