php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #76555 Unexpected default of phpdbg.eol ini setting
Submitted: 2018-06-30 11:20 UTC Modified: 2021-11-15 16:50 UTC
Votes:1
Avg. Score:3.0 ± 0.0
Reproduced:0 of 0 (0.0%)
From: cmb@php.net Assigned: cmb (profile)
Status: Not a bug Package: phpdbg
PHP Version: 7.2.7 OS: *
Private report: No CVE-ID: None
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: cmb@php.net
New email:
PHP Version: OS:

 

 [2018-06-30 11:20 UTC] cmb@php.net
Description:
------------
Apparently, the ini setting phpdbg.eol defaults to "2"[1] which
corresponds to PHPDBG_EOL_CR[2] which appears to be very uncommon
nowadays.  Is this perhaps a related to the out-commented
PHPDBG_EOL_LFCR[3], and the default is actually supposed to be
"1"?

Even more confusing, setting phpdbg.eol to a numeric value is
apparently not allowed, see test script.  I would either expect
that this is allowed, or that the setting would default to "CR"
instead of "2".

Anatol, since you introduced this ini setting[4], could you please
have a look at this?

[1] <https://github.com/php/php-src/blob/PHP-7.2.7/sapi/phpdbg/phpdbg.c#L74>
[2] <https://github.com/php/php-src/blob/PHP-7.2.7/sapi/phpdbg/phpdbg_eol.h#L34>
[3] <https://github.com/php/php-src/blob/PHP-7.2.7/sapi/phpdbg/phpdbg_eol.h#L32>
[4] <http://git.php.net/?p=php-src.git;a=commit;h=e011f0ca60a7d4d062b559dbbf7cfabef8f7a75b>

Test script:
---------------
<?php

var_dump(ini_get('phpdbg.eol'));
var_dump(ini_set('phpdbg.eol', "2"));
var_dump(ini_get('phpdbg.eol'));
var_dump(ini_set('phpdbg.eol', 'CR'));
var_dump(ini_get('phpdbg.eol'));


Actual result:
--------------
string(1) "2"
bool(false)
string(1) "2"
string(1) "2"
string(2) "CR"

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2018-06-30 11:21 UTC] cmb@php.net
-Status: Open +Status: Assigned -Assigned To: +Assigned To: ab
 [2021-11-15 16:50 UTC] cmb@php.net
-Status: Assigned +Status: Not a bug -Assigned To: ab +Assigned To: cmb
 [2021-11-15 16:50 UTC] cmb@php.net
Given that the behavior is properly documented[1], and that the
setting is removed as of PHP 8.1.0[2], it makes no sense to keep
this ticket open.

[1] <https://www.php.net/manual/en/phpdbg.configuration.php>
[2] <https://github.com/php/php-src/pull/7147>
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Tue Mar 19 04:01:31 2024 UTC