Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tests/bug73055.phpt fails with PHP 7.4.0 #99

Closed
Whissi opened this issue Nov 28, 2019 · 2 comments
Closed

tests/bug73055.phpt fails with PHP 7.4.0 #99

Whissi opened this issue Nov 28, 2019 · 2 comments

Comments

@Whissi
Copy link
Contributor

Whissi commented Nov 28, 2019

---- EXPECTED OUTPUT
Test
%r(exception ')?%rhttp\Exception\BadQueryStringException%r(' with message '|: )%rhttp\QueryString::__construct(): Max input nesting level of %d exceeded%r'?%r in %sbug73055.php:%d
Stack trace:
#0 %sbug73055.php(%d): http\QueryString->__construct('[[[[[[[[[[[[[[[...')
#1 {main}
===DONE===
---- ACTUAL OUTPUT
Test
http\Exception\BadQueryStringException: http\QueryString::__construct(): Max input nesting level of 64 exceeded in /var/tmp/portage/dev-php/pecl-http-3.2.3/work/php7.4/tests/bug73055.php:6
Stack trace:
#0 /var/tmp/portage/dev-php/pecl-http-3.2.3/work/php7.4/tests/bug73055.php(6): http\QueryString->__construct()
#1 {main}
===DONE===
---- FAILED
@m6w6
Copy link
Owner

m6w6 commented Dec 2, 2019

Thank you for this report. Unfortunately you did not provide enough information to handle this bug, while the tests succeed on travis-ci: https://travis-ci.org/m6w6/ext-http/jobs/618250434

@m6w6 m6w6 closed this as completed Dec 2, 2019
@Whissi
Copy link
Contributor Author

Whissi commented Dec 2, 2019

This test depends on zend.exception_ignore_args = Off whereas I was testing with zend.exception_ignore_args = On. Could you please adjust test suite to enforce correct setting?

m6w6 added a commit that referenced this issue Dec 4, 2019
Tests were failing on systems with zend.exception_ignore_arg set.
Thanks to Thomas Deutschmann (@Whissi)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants