New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use Monolog:SyslogHandler syslog default facility #17855
Conversation
Issue#9400
Avoid null in ErrorLogHandler constructor call: Use level param; default others
@mwithheld I think you'll need to merge these changes w/ 4.x-dev to remove the conflicts, something like:
|
plugins/Monolog/config/config.php
Outdated
@@ -105,7 +105,8 @@ | |||
->method('setFormatter', DI\get('log.lineMessageFormatter.file')), | |||
|
|||
'\Monolog\Handler\SyslogHandler' => DI\create() | |||
->constructor(DI\get('log.syslog.ident'), 'syslog', DI\get('log.level.syslog')) | |||
->constructorParameter('ident' => DI\get('log.syslog.ident')) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
->constructorParameter('ident' => DI\get('log.syslog.ident')) | |
->constructorParameter('ident', DI\get('log.syslog.ident')) |
plugins/Monolog/config/config.php
Outdated
@@ -105,7 +105,8 @@ | |||
->method('setFormatter', DI\get('log.lineMessageFormatter.file')), | |||
|
|||
'\Monolog\Handler\SyslogHandler' => DI\create() | |||
->constructor(DI\get('log.syslog.ident'), 'syslog', DI\get('log.level.syslog')) | |||
->constructorParameter('ident' => DI\get('log.syslog.ident')) | |||
->constructorParameter('level' => DI\get('log.level.syslog')) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
->constructorParameter('level' => DI\get('log.level.syslog')) | |
->constructorParameter('level', DI\get('log.level.syslog')) |
@mwithheld left a couple suggestions, after those I should be able to merge it. |
This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers |
Hi @mwithheld, sorry for the late reply, it seems like you added some commits for another PR into this one (the one dealing w/ NotSupportedBrowserException). |
Good catch - I've reset the branch to before the incorrect ones, force-pushed the branch, and merged updates from the 4.x-dev branch. Hopefully that is clean now. |
Thanks for fixing up the PR @mwithheld! It's now merged. |
Description:
This is a further fix to #17764, fixing the Monolog:SyslogHandler create() call to use its constructor-default facility rather than manually setting it to 'syslog' incorrectly.