Security and core updates for PHP 8.5 and s6-overlay v3#13
Open
metalcated wants to merge 4 commits intoOrganizr:masterfrom
Open
Security and core updates for PHP 8.5 and s6-overlay v3#13metalcated wants to merge 4 commits intoOrganizr:masterfrom
metalcated wants to merge 4 commits intoOrganizr:masterfrom
Conversation
metalcated
commented
Feb 13, 2026
metalcated
commented
- Upgrade s6-overlay from v2.2.0.3 to v3.2.1.0
- Add s6 v3 compatibility symlinks (with-contenv, execlineb)
- Add php symlink to php85 for compatibility
- Fix PHP 8.5 socket naming consistency
- Fix execute permissions on init and service scripts
- Upgrade s6-overlay from v2.2.0.3 to v3.2.1.0 - Add s6 v3 compatibility symlinks (with-contenv, execlineb) - Add php symlink to php85 for compatibility - Fix PHP 8.5 socket naming consistency - Fix execute permissions on init and service scripts
Required for JWT token validation with newer versions of lcobucci/jwt that use sodium for signature verification.
Roxedus
requested changes
Feb 13, 2026
Member
Roxedus
left a comment
Roxedus
left a comment
There was a problem hiding this comment.
I will not accept a half implemented S6v3 transition.
- Update all init scripts to use /command/with-contenv shebang - Update all service scripts to use /command/with-contenv shebang - Remove compatibility symlinks from Dockerfiles (no longer needed) - Add support for child images to provide /defaults/php-local.ini
Author
|
You're right man. Half ass with the symlinks. Fixed. |
Member
|
With s6v3 there's no guaranteed execution order for services in |
Replace cont-init.d and services.d with proper s6-rc.d structure: - init-adduser -> init-config -> init-setup -> init-debug -> init-services - svc-nginx, svc-php-fpm, svc-cron depend on init-services - Explicit dependency declarations instead of lexical ordering
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.