Closed (fixed)
Project:
Monolog
Version:
2.0.2
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
5 Jul 2022 at 07:23 UTC
Updated:
2 Sep 2022 at 08:32 UTC
Jump to comment: Most recent
Problem/Motivation
We're extending the Logger class from Monolog just to provide a map between RFC 5424 levels and Monolog levels. Seldaek here suggest to move this map into Monolog itself.
Steps to reproduce
This issue has been mitigated by #3284825: Argument add to addRecord upstream, but we need to prevent that problem like this will occur in future.
Proposed resolution
Remove the Logger class from this module and move the level map upstream.
Remaining tasks
Write a patch for the module and the library.
User interface changes
None
API changes
None
Data model changes
None
Issue fork monolog-3294236
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #2
lussolucaComment #3
lussolucaComment #11
lussolucaCommitted and pushed to 2.x and 3.0.x
Comment #16
upchuk commentedThis breaks BC I'm afraid. I opened another issue: https://www.drupal.org/project/monolog/issues/3307496