Req->Bug #78656 [Asn]: Parse errors classified as highest log-level
| From: | cmb@php.net | Date: | Wed, 09 Oct 2019 15:26:10 +0000 |
| Subject: | Req->Bug #78656 [Asn]: Parse errors classified as highest log-level | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-223165@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=78656&edit=1
ID: 78656
Updated by: cmb@php.net
Reported by: erik at coretech dot se
Summary: Parse errors classified as highest log-level
Status: Assigned
-Type: Feature/Change Request
+Type: Bug
Package: Scripting Engine problem
Operating System: CentOS 7
PHP Version: 7.3.10
Assigned To: cmb
Block user comment: N
Private report: N
New Comment:
Grr, hit that bug again.
Previous Comments:
------------------------------------------------------------------------
[2019-10-09 15:23:16] cmb@php.net
Reclassifying as bug, see <https://github.com/php/php-src/pull/4805#issuecomment-540004297>ff.
------------------------------------------------------------------------
[2019-10-09 14:30:22] erik at coretech dot se
The following pull request has been associated:
Patch Name: Request #78656 Parse errors classified as highest log-level
On GitHub: https://github.com/php/php-src/pull/4805
Patch: https://github.com/php/php-src/pull/4805.patch
------------------------------------------------------------------------
[2019-10-09 12:52:45] sjon@php.net
I agree - this setup (by default) will cause any parse-error to be broadcasted (using wall) to all
users on a shared development machine
------------------------------------------------------------------------
[2019-10-09 11:53:12] erik at coretech dot se
Description:
------------
Parse errors are classified as the highest log level (LOG_EMERG). This is problem since
Centos/Fedora writes LOG_EMERG messages to all users:
https://src.fedoraproject.org/rpms/rsyslog/blob/master/f/rsyslog.conf#_59
LOG_EMERG is described as: "system is unusable". A php-script with a parse error would not
in most cases render the system unstable. At most LOG_ERR or LOG_CRIT would be more suitable. Fatal
errors are classified as LOG_ERR. Why not make parse errors the same level?
The source:
https://github.com/php/php-src/blob/PHP-7.3/main/main.c#L1300
The commit that changed this:
https://github.com/php/php-src/commit/3edf7d960cb14f2ee45944e7c567af92af25bda1
------------------------------------------------------------------------
--
Edit this bug report at https://bugs.php.net/bug.php?id=78656&edit=1