mirror of
https://github.com/Seldaek/monolog.git
synced 2026-04-26 08:05:53 +03:00
[GH-ISSUE #170] FingersCrossedHandler does not use registered processors #56
Labels
No labels
Bug
Documentation
Feature
Needs Work
Support
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/monolog#56
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @tpetry on GitHub (Mar 11, 2013).
Original GitHub issue: https://github.com/Seldaek/monolog/issues/170
Processors registered to FingersCrossedHandler will not be included in any logged information because FingersCrossedHandler extends AbstractHandler and not AbstractProcessingHandler.
But it's nowhere documented (not in documentation nor in the sourcecode) that FingersCrossedHandler will not run the log message through registered processors. They are added and silently ignored. I don't know whether it's expected behaviour (then adding an processor should throw an exception) or simply an bug.
@stof commented on GitHub (Mar 11, 2013):
the same is true for all other handlers forwarding to record to a nested handler.
@Seldaek commented on GitHub (Mar 11, 2013):
Indeed BufferHandler, FingersCrossedHandler, GroupHandler and NullHandler are affected. The latter is not an issue really. The others should be fixed. One note though, in the case of the FingersCrossedHandler you really should put the processors after it to avoid wasting cycles on processing log records that will never be stored, but that's a detail.