Bug #78656 [Asn->Csd]: Parse errors classified as highest log-level

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 +Status: Closed 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: 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 Previous Comments: ------------------------------------------------------------------------ [2019-10-09 15:26:10] cmb@php.net Grr, hit that bug again. ------------------------------------------------------------------------ [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