Matomo application logs can now be written in syslog and errorlog #17764

Description:

Issue #9400 Add log handlers syslog and errorlog; Add config for syslog ident; Document options.

Review

• [ ] Functional review done
• [ ] Potential edge cases thought about (behavior of the code with strange input, with strange internal state or possible interactions with other Matomo subsystems)
• [ ] Usability review done (is anything maybe unclear or think about anything that would cause people to reach out to support)
• [ ] Security review done see checklist
• [ ] Code review done
• [ ] Tests were added if useful/possible
• [ ] Reviewed for breaking changes
• [ ] Developer changelog updated if needed
• [ ] Documentation added if needed
• [ ] Existing documentation updated if needed

This is an update to PR #17745

@mwithheld Thanks for fixing the issues! I left one other comment. After it's addressed it should be good to merge.

Added commit 14992a8 ErrorLogHandler constructor: Avoid null

Good catch - the discussion is welcome :) That was not intended - I'm just
not used to using GitHub for editing PRs.

On Thu, Jul 15, 2021 at 7:20 PM dizzy @.***> wrote:

@.**** commented on this pull request.

In plugins/Monolog/config/config.php
https://github.com/matomo-org/matomo/pull/17764#discussion_r670919393:

@@ -97,6 +99,13 @@
'Piwik\Plugins\Monolog\Handler\FileHandler' => DI\create()
->constructor(DI\get('log.file.filename'), DI\get('log.level.file'))
->method('setFormatter', DI\get('log.lineMessageFormatter.file')),
+

• '\Monolog\Handler\ErrorLogHandler' => DI\create()
• ->constructorParameter('level', DI\get('log.level.errorlog'))
• ->method('setFormatter', DI\get('log.lineMessageFormatter.file')),
• '\Monolog\Handler\SyslogHandler' => DI\create()
• ->constructor(DI\get('log.syslog.ident'), 'syslog', DI\get('log.level.syslog')),

Apologies for the back and forth, but one last question here: was it
intentional not to use ->method('setFormatter',
DI\get('log.lineMessageFormatter.file')) here?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/matomo-org/matomo/pull/17764#pullrequestreview-707953192,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGCSOXPWKB6RSVBKTMBCRDTX6JQRANCNFSM5AKBNYHA
.

@mwithheld I've merged the PR, thanks for going through the trouble of creating the PR and following up on it! The contribution is greatly appreciated.

@diosmosis could you please document this new capability in https://matomo.org/faq/troubleshooting/faq_115/ ? Thanks

documented

Hi @mwithheld, thanks for creating the change so quickly! Would you be able to put it in a new pull request? This one has already been merged so it's no longer useful to push here.

Ok, thanks! And thanks for adding syslog support in the first place. Obviously, I was waiting for it =)