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

ChromePHP/FirePHPHandler json_encode recursion #160

Closed
peshi opened this issue Feb 11, 2013 · 3 comments
Closed

ChromePHP/FirePHPHandler json_encode recursion #160

peshi opened this issue Feb 11, 2013 · 3 comments

Comments

@peshi
Copy link

peshi commented Feb 11, 2013

Using ChromePHPHandler or FirePHPHandler blows my application, generating a 3gb logfile with the same error.

I'm using Symfony 2.2.x RC2 with FOSUserBundle, this is the registerAction.
I had to uncomment the handlers in config_dev.yml

The application is working just fine in Syfmony 2.1.x

[Mon Feb 11 01:45:05 2013] [error] [client 10.0.0.2] PHP Warning: json_encode(): recursion detected in /var/www/myproject/vendor/monolog/monolog/src/Monolog/Handler/ChromePHPHandler.php on line 100, referer: http://mydomain/app_dev.php/en/login

I can provide a complete stack trace upon request.

@stof
Copy link
Contributor

stof commented Feb 11, 2013

are you using the latest version of Monolog ? This should have been fixed in 63e3bfd

@stof
Copy link
Contributor

stof commented Feb 11, 2013

hmm, it seems that the ChromePHPHandler has been missed in this commit

@peshi
Copy link
Author

peshi commented Feb 11, 2013

Perfect, thanks!

Ok - I've got 15+ deprecation warnings. This is working perfectly in Firefox (it's splitting headers into chunks).
Chrome gives me the following error.

Error 325 (net::ERR_RESPONSE_HEADERS_TOO_BIG): Unknown error.

This is probably not a monolog issue, but i'll write about it here so that you are aware of the problem.

ccampbell/chromephp#12

http://stackoverflow.com/questions/3326210/can-http-headers-be-too-big-for-browsers

mathroc pushed a commit to mathroc/monolog that referenced this issue Dec 30, 2013
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

3 participants