Bug #79178 [Com]: [parse_url] underscore at end of host

Edit report at https://bugs.php.net/bug.php?id=79178&edit=1 ID: 79178 Comment by: nawarian at gmail dot com Reported by: progr-d at yandex dot ru Summary: [parse_url] underscore at end of host Status: Open Type: Bug Package: *URL Functions Operating System: macOS 10.13.6 PHP Version: 7.3.14 Block user comment: N Private report: N New Comment: While parsing many parts of the url (scheme, user, pass, host, fragment, query, path) php behaves this way: whenever it finds parts of the string being a control character (cctype::iscntrl) it replaces such character with an underscore. Why? I don't know... I've submitted a pull request fixing the behaviour mentioned on "host" part only. But such issue will keep happening on every url part I've mentioned before. I wouldn't mind fixing those as well, but sounds like a bigger change that affects the API. Thoughts? Previous Comments: ------------------------------------------------------------------------ [2020-03-01 00:24:38] nawarian at gmail dot com The following pull request has been associated: Patch Name: Fix #79178 parse_url check control chars on host On GitHub: https://github.com/php/php-src/pull/5225 Patch: https://github.com/php/php-src/pull/5225.patch ------------------------------------------------------------------------ [2020-01-27 19:22:02] progr-d at yandex dot ru Description: ------------ When processing a file through function file, I met this feature, which I could not have expected in this place. Yes, I know that you need to use flag FILE_IGNORE_NEW_LINES, but having received an underscore, for a long time I could not understand where it came from. Test script: --------------- var_dump(parse_url('http://yandex.ru' . PHP_EOL, PHP_URL_HOST)); Expected result: ---------------- string(10) "yandex.ru" or Exception Actual result: -------------- string(10) "yandex.ru_" ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=79178&edit=1