8000 User definable duplication for DeduplicationHandler · Issue #1878 · Seldaek/monolog · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
User definable duplication for DeduplicationHandler #1878
Closed
@cracksalad

Description

@cracksalad

The DeduplicationHandler defines duplication as multiple records having the same message and level. In general this is probably the right way to do it, but in specific cases it might be required to include some other attributes like context, e 5C98 xtra or channel in this condition as well.
Therefore you would want to overwrite the DeduplicationHandler::isDuplicate() method to define duplication by yourself. This is currently impossible due to the method being private. That is why I suggest to change the visibility to protected.
Additionally it would probably be desirable to move the duplicationStore stuff to another method and pass the $store variable to the isDuplicate() method. Just to make it a bit simpler to overwrite the method.

Of course one could also (as a workaround) add the additional attributes from say context to the message itself, but that would change the outcome in your logfiles or what have you.

If you approve this suggestion, I am willing to help and perform those changes by myself and create a PR afterwards. Just let me know!

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      0