starred/monolog
1
0
Fork 0
mirror of https://github.com/Seldaek/monolog.git synced 2026-04-27 00:25:50 +03:00
Code Issues 31 Projects Releases 5 Packages Wiki Activity

[GH-ISSUE #1054] StreamHandler: Set $useLocking to true by default #428

New issue
Closed
opened 2026-03-04 02:14:57 +03:00 by kerem · 1 comment
kerem commented 2026-03-04 02:14:57 +03:00
Owner
Copy link

Originally created by @tomsommer on GitHub (Sep 13, 2017).
Original GitHub issue: https://github.com/Seldaek/monolog/issues/1054

Would it not make sense to set $useLocking to true by default? Otherwise you risk random concurrency/permission errors.

github.com/Seldaek/monolog@7405bb5d67/src/Monolog/Handler/StreamHandler.php (L39)

github.com/Seldaek/monolog@07b6c1c417/src/Monolog/Handler/RotatingFileHandler.php (L45)

Originally created by @tomsommer on GitHub (Sep 13, 2017). Original GitHub issue: https://github.com/Seldaek/monolog/issues/1054 Would it not make sense to set $useLocking to true by default? Otherwise you risk random concurrency/permission errors. https://github.com/Seldaek/monolog/blob/7405bb5d673df9ee05698876843e85b0b8a599d4/src/Monolog/Handler/StreamHandler.php#L39 https://github.com/Seldaek/monolog/blob/07b6c1c4171a030787f2bc961272b1887032304e/src/Monolog/Handler/RotatingFileHandler.php#L45
kerem closed this issue 2026-03-04 02:14:57 +03:00
kerem commented 2026-03-04 02:14:58 +03:00
Author
Owner
Copy link

@Seldaek commented on GitHub (Jun 17, 2018):

As per https://github.com/Seldaek/monolog/issues/379 - this is only an issue if you log long lines or in very odd setups. For most use cases locking isn't needed even with high concurrency, so I don't think it's a good idea to enable by default.

<!-- gh-comment-id:397884835 --> @Seldaek commented on GitHub (Jun 17, 2018): As per https://github.com/Seldaek/monolog/issues/379 - this is only an issue if you log long lines or in very odd setups. For most use cases locking isn't needed even with high concurrency, so I don't think it's a good idea to enable by default.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
esfix
2.x
1.x
3.10.0
2.11.0
3.9.0
3.8.1
3.8.0
2.10.0
3.7.0
3.6.0
2.9.3
3.5.0
2.9.2
3.4.0
3.3.1
2.9.1
3.3.0
2.9.0
3.2.0
2.8.0
3.1.0
2.7.0
1.27.1
3.0.0
2.6.0
3.0.0-RC1
2.5.0
2.4.0
1.27.0
2.3.5
2.3.4
2.3.3
2.3.2
2.3.1
2.3.0
1.26.1
2.2.0
1.26.0
2.1.1
1.25.5
2.1.0
1.25.4
2.0.2
1.25.3
2.0.1
1.25.2
1.25.1
1.25.0
2.0.0
2.0.0-beta2
2.0.0-beta1
1.24.0
1.23.0
1.22.1
1.22.0
1.21.0
1.20.0
1.19.0
1.18.2
1.18.1
1.18.0
1.17.2
1.17.1
1.17.0
1.16.0
1.15.0
1.14.0
1.13.1
1.13.0
1.12.0
1.11.0
1.10.0
1.9.1
1.9.0
1.8.0
1.7.0
1.6.0
1.5.0
1.4.1
1.4.0
1.3.1
1.3.0
1.2.1
1.2.0
1.1.0
1.0.2
1.0.1
1.0.0
1.0.0-RC1
Labels
Clear labels   
Bug

   
Documentation

   
Feature

   
Needs Work

   
Support

   
pull-request

Mirrored from GitHub Pull Request

No labels
Bug
Documentation
Feature
Needs Work
Support
pull-request
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/monolog#428
No description provided.