Edit report at https://bugs.php.net/bug.php?id=66239&edit=1
ID: 66239
Updated by: remi@php.net
Reported by: sjon at hortensius dot net
Summary: Fpm won't error_log to syslog, php works fine
-Status: Open
+Status: Closed
Type: Bug
Package: FPM related
Operating System: archlinux
PHP Version: 5.5.6
-Assigned To:
+Assigned To: remi
Block user comment: N
Private report: N
New Comment:
The fix for this bug has been committed.
Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
For Windows:
http://windows.php.net/snapshots/
Thank you for the report, and for helping us make PHP better.
Fixed, see #67531
Previous Comments:
------------------------------------------------------------------------
[2013-12-06 11:59:58] krakjoe@php.net
I think the first patch is correct, I don't see when you would want syslog expanded (usually to
a non-existent path) in any case ...
Alternate patch attached anyway ...
------------------------------------------------------------------------
[2013-12-06 11:56:56] krakjoe@php.net
The following patch has been added/updated:
Patch Name: fpm-syslog-bug.patch
Revision: 1386331016
URL: https://bugs.php.net/patch-display.php?bug=66239&patch=fpm-syslog-bug.patch&revision=1386331016
------------------------------------------------------------------------
[2013-12-06 11:55:37] krakjoe@php.net
The following patch has been added/updated:
Patch Name: fpm-syslog-bug-alternate.patch
Revision: 1386330937
URL: https://bugs.php.net/patch-display.php?bug=66239&patch=fpm-syslog-bug-alternate.patch&revision=1386330937
------------------------------------------------------------------------
[2013-12-06 11:47:09] krakjoe@php.net
Why depend on wp->config->chroot ?
Can you think of a case where it is reasonable for "syslog" to be expanded ?
------------------------------------------------------------------------
[2013-12-06 09:52:20] tony2001@php.net
But why do you change fpm_evaluate_full_path()? It's used everywhere and
php_value/php_admin_value is just one use case.
I believe you should be changing the checks in this 'if' instead:
/* env[], php_value[], php_admin_values[] */
if (!wp->config->chroot) {
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
https://bugs.php.net/bug.php?id=66239
--
Edit this bug report at https://bugs.php.net/bug.php?id=66239&edit=1