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
/uphealth check from nginx access logs (access_log off; log_not_found off) - #60 — Scheduler supervisor now uses
schedule:workinstead of awhile/sleep/schedule:runloop - #38 — Fix
ssh-keygen"Too many arguments" by usingescapeshellarg()for all parameters - #59 — Clear
git_commit_shaafter deploying a specific commit so future deploys use HEAD - #67 — Container logging: generate
docker/php-fpm.confwithcatch_workers_output = yes, useerrorlogchannel (notstderr— which doesn't work with PHP-FPM), add logging guidance tocoolify: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:installshows logging recommendation - Verify generated Dockerfile includes
php-fpm.confCOPY line - Verify
coolify:provisionsetsLOG_STACK=daily,errorlog
+86
additions
-8
deletions
6
files changed