| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
There was a problem hiding this comment.
Thanks for opening this pull request! Be sure to follow the pull request template!
Sorry, something went wrong.
|
Any chance we get this merged? |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Description:
Run the BookStack async queue worker as the abc user using s6-setuidgid.
A corresponding changelog entry has also been added to readme-vars.yml.
Benefits of this PR and context:
The queue worker currently inherits root privileges from s6, while PHP-FPM runs as the abc user.
If the queue worker is the first process to write to /config/log/bookstack/laravel.log, such as during a transient Redis failure, the file is created as root:root with mode 0644. PHP-FPM cannot subsequently append to it, causing Laravel logging to throw an UnexpectedValueException and web requests to return HTTP 500 errors.
Running the queue worker as abc:
How Has This Been Tested?
Source / References:
AI assistance disclosure:
The issue investigation and preparation of this change were assisted by OpenAI Codex. The resulting diagnosis, code change, and validation results were reviewed by the contributor.