php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #78656 Parse errors classified as highest log-level
Submitted: 2019-10-09 11:53 UTC Modified: 2019-10-09 15:26 UTC
From: erik at coretech dot se Assigned: cmb (profile)
Status: Closed Package: Scripting Engine problem
PHP Version: 7.3.10 OS: CentOS 7
Private report: No CVE-ID: None
View Add Comment Developer Edit
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If this is not your bug, you can add a comment by following this link.
If this is your bug, but you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: erik at coretech dot se
New email:
PHP Version: OS:

 

 [2019-10-09 11:53 UTC] 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

Patches

Add a Patch

Pull Requests

Pull requests:

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2019-10-09 12:52 UTC] 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 14:30 UTC] 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 15:23 UTC] cmb@php.net
-Assigned To: +Assigned To: cmb
 [2019-10-09 15:23 UTC] cmb@php.net 
Reclassifying as bug, see <https://github.com/php/php-src/pull/4805#issuecomment-540004297>ff.
 [2019-10-09 15:26 UTC] cmb@php.net
-Type: Feature/Change Request +Type: Bug
 [2019-10-09 15:26 UTC] cmb@php.net 
Grr, hit that bug again.
 [2019-10-09 15:31 UTC] cmb@php.net 
Automatic comment on behalf of erik@coretech.se
Revision: http://git.php.net/?p=php-src.git;a=commit;h=3164186d53533ac5239790e692b33dd6e56c18df
Log: Fix #78656: Parse errors classified as highest log-level
 [2019-10-09 15:31 UTC] cmb@php.net
-Status: Assigned +Status: Closed
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Tue Apr 23 22:01:31 2024 UTC