Update monolog.rst

[zhil]

Default value for buffer_size is unlimited, which worth being mentioned

[zhil]

according to https://github.com/symfony/monolog-bundle/blob/master/DependencyInjection/Configuration.php default value is unlimited, not 30

[xabbuh]

looks like the XML example would need to be updated too

[zhil]

@xabbuh I have updated xml as well

[javiereguiluz]

👍

Status: reviewed

---

@zhil a very nice catch in a very important document. We must be extra-careful in the configuration references to not make any mistake.

[wouterj]

Thanks @zhil!

Fyi, I've merged this PR in the 2.7 version of the docs. I'll update the more recent versions from there.

[zhil]

@wouterj @javiereguiluz @xabbuh
probably, it would be nice to notice in documentation, that by defaults monolog stop buffering logs after first error (stop_buffering is true by default).
So, complicated console commands could generate gigabytes of logs easily - just fixed it on production.

should I create PR for it?

[xabbuh]

@zhil I think that would be a good idea. Are you still interested in creating a PR?

[zhil]

@xabbuh well, technically, its already documented
https://github.com/symfony/monolog-bundle/blob/master/DependencyInjection/Configuration.php#L93

IMHO "stop_buffering" is confusing term itself. If its set to true (default value) - finger_crossed in fact disabled after first error and everything after first error are placed in logs.

Tthats correct approach for websites, but could be wrong for command line scripts. (If script do a lot of work in loops, it could create much more logs, as expected).

I am not sure, where exactly should I add notice about it. Mayb short notice like "Please note, that by default fingers_crossed could create huge logs for command line scripts after first error"

[xabbuh]

@zhil Is that the only place where we document that option?

[zhil]

@xabbuh well, I suppose note could be also added ether in https://symfony.com/doc/current/logging/monolog_console.html or https://symfony.com/doc/current/logging.html

[xabbuh]

@zhil Yeah, I think the former one is a must have, in the latter one we can probably omit it to not confuse the reader with too many details.