This change symlinks the default PHP error log file to /dev/stderr so that docker logs will also include PHP errors from the airship application container. Currently, only Apache routes logs to stderr.
Note: this fix will break on major or minor (x or x.x) PHP version changes (because the log file path will change). If this is a concern, then I'll find ways to fix it before this gets merged.
Issues Addressed (Optional)
94 Docker PHP Logs
Contributor Agreement (Required)
I am submitting this pull request under one or more of the following
licenses:
Furthermore, I understand that CMS Airship is released under the GNU Public
License to the general public, as well as private commercial licenses
(purchasable from Paragon Initiative Enterprises).
By submitting this pull request, I acknowledge that my contribution will be
incorporated into CMS Airship, and consent for it to be handled as outlined
above.
(This does not in any way restrict your rights to use your own modifications.
The purpose of this agreement is to maximize awareness and transparency.)
Summary
This change symlinks the default PHP error log file to /dev/stderr so that
docker logswill also include PHP errors from the airship application container. Currently, only Apache routes logs to stderr.Note: this fix will break on major or minor (x or x.x) PHP version changes (because the log file path will change). If this is a concern, then I'll find ways to fix it before this gets merged.
Issues Addressed (Optional)
94 Docker PHP Logs
Contributor Agreement (Required)
I am submitting this pull request under one or more of the following licenses:
Furthermore, I understand that CMS Airship is released under the GNU Public License to the general public, as well as private commercial licenses (purchasable from Paragon Initiative Enterprises).
By submitting this pull request, I acknowledge that my contribution will be incorporated into CMS Airship, and consent for it to be handled as outlined above.
(This does not in any way restrict your rights to use your own modifications. The purpose of this agreement is to maximize awareness and transparency.)
Labels Requested
none