Page MenuHomePhabricator

Fix MWLoggerMonologHandler::isHandling() to work with Monolog 1.12.0
Closed, ResolvedPublic

Description

The current isHandling() implementation expects a full log record so it can use channel and the context to decide if a record will be emitted based on the legacy logging rules when $useLegacyFilter is true. Monolog 1.12.0 made a change that only passes the log level in the record provided to isHandling() so we need to adjust our implementation.

Event Timeline

bd808 claimed this task.
bd808 raised the priority of this task from to Medium.
bd808 updated the task description. (Show Details)
bd808 added subscribers: gerritbot, Legoktm, bd808 and 2 others.

Change 190618 had a related patch set uploaded (by BryanDavis):
Update MWLoggerMonologHandler for Monolog 1.12.0

https://gerrit.wikimedia.org/r/190618

Patch-For-Review

Change 190618 merged by jenkins-bot:
Update MWLoggerMonologHandler for Monolog 1.12.0

https://gerrit.wikimedia.org/r/190618