Skip to content

feat(php-lol): run nginx + php-fpm rootless#317

Draft
Copilot wants to merge 3 commits into
masterfrom
copilot/update-nginx-config-non-root
Draft

feat(php-lol): run nginx + php-fpm rootless#317
Copilot wants to merge 3 commits into
masterfrom
copilot/update-nginx-config-non-root

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 30, 2026

  • Add .rootless marker file to dockerfiles/php-lol/
  • Update action.yml to detect the marker and append -rootless suffixed tags

Copilot AI and others added 2 commits March 30, 2026 09:42
- Remove 'user www-data' directive from nginx.conf (not needed for rootless)
- Move PID files to /tmp for nginx, php-fpm, and supervisord
- Change nginx listen port from 80 to 8080 (non-root can't bind < 1024)
- Remove 'user = root' from supervisord.conf and app.conf
- Set php-fpm pool user/group to nobody:nogroup
- Change /var/cache/nginx and /var/lib/php/session ownership to 65534:65534
- Add USER 65534:65534 directive in all 4 Dockerfiles
- Update EXPOSE from 80/443 to 8080
- Update test checksums for modified config files

Agent-Logs-Url: https://github.com/bdossantos/dockerfiles/sessions/4a2f865b-f62a-4d89-92d7-df6224b6e8c6

Co-authored-by: bdossantos <245284+bdossantos@users.noreply.github.com>
Copilot AI requested a review from bdossantos May 12, 2026 09:16
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