Skip to content
This repository was archived by the owner on Nov 27, 2020. It is now read-only.

made browser logs commented by defaults (refs symfony/symfony#8413) #569

Merged
merged 1 commit into from
Jul 8, 2013

Conversation

fabpot
Copy link
Member

@fabpot fabpot commented Jul 3, 2013

FirePHP and ChromePHP are now disabled by default as they can make the application broke if the web server is not configured with large header sizes.

see symfony/symfony#8413

@stof
Copy link
Member

stof commented Jul 3, 2013

FirePHP should not break it as it uses a separate header for each log entry, so the header does not become too big. ChromePHP indeed makes the heade size grow as it uses a single header.

type: chromephp
level: info
# uncomment to get logging in your browser
# you may have to allow bigger header sizes in your Web server configuration
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo: web lowercase

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think nginx should be mentioned specifically in the comment. What other web servers require this configuration?

@Tobion
Copy link
Contributor

Tobion commented Jul 3, 2013

I think both handlers should not be enabled by default because the feature should be opt-in and not opt-out.
Even if FirePHP does not break header size.

@apfelbox
Copy link

apfelbox commented Jul 3, 2013

See the referenced issue. Only ChromePHP breaks without tuning the configuration of nginx. But I'm fine disabling both.

@lyrixx
Copy link
Member

lyrixx commented Jul 6, 2013

+1 I got this issue so much time...

fabpot added a commit that referenced this pull request Jul 8, 2013
This PR was merged into the master branch.

Discussion
----------

made browser logs commented by defaults (refs symfony/symfony#8413)

FirePHP and ChromePHP are now disabled by default as they can make the application broke if the web server is not configured with large header sizes.

see symfony/symfony#8413

Commits
-------

262af2d made browser logs commented by defaults (refs #8413)
@fabpot fabpot merged commit 262af2d into master Jul 8, 2013
@Tobion
Copy link
Contributor

Tobion commented Aug 27, 2013

@fabpot you can remove the monolog-tweaks branch from the repo

@fabpot fabpot deleted the monolog-tweaks branch August 27, 2013 17:36
@bamarni
Copy link
Contributor

bamarni commented Oct 19, 2013

Should it also be backported to 2.3 (as it's the LTS version)?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants