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.
Description
Description
Details
Details
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
Update MWLoggerMonologHandler for Monolog 1.12.0 | mediawiki/core | master | +12 -10 |
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Resolved | Anomie | T1272 Deploy ApiFeatureUsage extension on WMF wikis | |||
Resolved | bd808 | T87521 Convert JobRunner.php to PSR-3 logging and add levels | |||
Resolved | bd808 | T88732 Decouple logging infrastructure failures from MediaWiki logging | |||
Resolved | bd808 | T88870 Prototype Monolog and rsyslog configuration to ship log events from MediaWiki to Logstash | |||
Resolved | bd808 | T85535 Upgrade monolog to 1.12.0 | |||
Resolved | bd808 | T89313 Fix MWLoggerMonologHandler::isHandling() to work with Monolog 1.12.0 |
Event Timeline
Comment Actions
Change 190618 had a related patch set uploaded (by BryanDavis):
Update MWLoggerMonologHandler for Monolog 1.12.0
Comment Actions
Change 190618 merged by jenkins-bot:
Update MWLoggerMonologHandler for Monolog 1.12.0