Seldaek / monolog

Sends your logs to files, sockets, inboxes, databases and various web services

Home Page:https://seldaek.github.io/monolog/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

User definable duplication for DeduplicationHandler

cracksalad opened this issue · comments

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, extra 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!