Implement monolog addon as LoggerFactory #1598

Merged
MrPetovan merged 6 commits from Art4/friendica-addons:loggerfactory-in-monolog-addon into develop 2025-02-08 04:16:48 +01:00
Contributor

Since https://github.com/friendica/friendica/pull/14682 Friendica has a new LoggerManager, that can be customized with a \Friendica\Core\Logger\Factory\LoggerFactory implementation.

This PR implements the Monolog factory as LoggerFactory and makes the current strategy config file obsolet. I also have added a READMD.md to describe how to switch to monolog and have bumped the version to 1.1.

Since https://github.com/friendica/friendica/pull/14682 Friendica has a new `LoggerManager`, that can be customized with a `\Friendica\Core\Logger\Factory\LoggerFactory` implementation. This PR implements the Monolog factory as `LoggerFactory` and makes the current strategy config file obsolet. I also have added a `READMD.md` to describe how to switch to monolog and have bumped the version to 1.1.
Art4 added 3 commits 2025-02-05 16:28:54 +01:00
Art4 changed title from loggerfactory-in-monolog-addon to Implement monolog addons as LoggerFactory 2025-02-05 16:30:44 +01:00
Art4 changed title from Implement monolog addons as LoggerFactory to Implement monolog addon as LoggerFactory 2025-02-05 16:31:00 +01:00
Art4 added 1 commit 2025-02-05 16:32:32 +01:00
Art4 added 1 commit 2025-02-06 10:27:28 +01:00
Art4 added 1 commit 2025-02-06 12:35:17 +01:00
MrPetovan added the
2025.02
label 2025-02-07 14:04:51 +01:00
MrPetovan reviewed 2025-02-07 14:07:36 +01:00
@ -0,0 +48,4 @@
$logger->pushProcessor(new UidProcessor());
$logger->pushProcessor(new IntrospectionProcessor($this->introspection, LogLevel::DEBUG));
$logfile = $this->config->get('system', 'logfile');
Owner

Why change the capitalization of $logLevel but not of $logfile?

Why change the capitalization of `$logLevel` but not of `$logfile`?
Author
Contributor

$logLevel comes from the LoggerFactory interface, but I had no reason to change $logfile.

`$logLevel` comes from the `LoggerFactory` interface, but I had no reason to change `$logfile`.
MrPetovan marked this conversation as resolved
MrPetovan approved these changes 2025-02-08 04:16:43 +01:00
MrPetovan merged commit 4454ce5883 into develop 2025-02-08 04:16:48 +01:00
Art4 deleted branch loggerfactory-in-monolog-addon 2025-02-08 13:39:42 +01:00
Sign in to join this conversation.
No description provided.