Actions
Epic #94356
closedEmbrace PSR-3
Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2021-06-11
Due date:
% Done:
100%
Estimated time:
(Total: 0.00 h)
Sprint Focus:
Description
General:
- Part 1: Get rid of everything that does not use psr-3 logging currently.
- Part 2 & 3: Refine our logging implementation and configuration, adapt ext:belog.
This epic is about Part 1 for now. Main goal: writing to sys_log only via DatabaseWriter, all other implementations should fall (deprecated), no manual querying of sys_log anymore like in DataHandler (anywhere else?), except ext:belog. Probably keep ext:belog and sys_log schema as is, for now.
- (#94315): Move to proper PSR-3 logging format.
- Transition all places that write to sys_log to use PSR-3 logger instead, configure DatabaseWriter for them
- DataHandler: This one uses be_user->writelog() via log() and newlog(). Look into these calls, use logger instead and also have a look at classes that may call DataHandler->log() and DataHandler->newlog() (they're public and I'd bet things like RelationHandler may use that). Configure DataHandler logger to use DatabaseWriter, and adapt DatabaseWriter to de-normalize towards the existing fields log() and newlog() wrote to.
- DataHandler: Consider additionally configuring memoryWriter to gather log rows of one request. See printLogErrorMessages() which currently queries sys_log to turn errors into flash messages if in web context. This could be fetched from MemoryWriter instead?
- Adapt Other places that call be_user->writelog
- Deprecate be_user->writelog
- Configure DatabaseWriter for ExceptionHandler and drop the manual insert query to sys_log in AbstractExceptionHandler. Keep an eye on bootstrap: What happens if a parse error happens in early bootstrap due to a broken (3rd party) extension? Is logging properly initialized and it's still logged to sys_log in this case?
- See if other places insert / select sys_log, adapt, eventually deprecate them.
Updated by Larry Garfield over 3 years ago
- Related to Bug #94043: Incorrect usage of PSR-3 logger added
Updated by Christian Kuhn over 3 years ago
- Related to Feature #82483: Introduce FingersCrossed LogWriter added
Updated by Christian Kuhn over 3 years ago
- Related to Feature #85506: Integrate monolog as logging framework added
Updated by Christian Kuhn over 3 years ago
- Related to Task #94344: Move PSR-3 logger to its own table added
Updated by Christian Kuhn about 2 years ago
- Status changed from New to Closed
The topic is stuck again and needs another restart. I'm closing here for the time being.
Updated by Christian Weiske 10 months ago
- Related to Bug #103799: Scheduler task exception error message misses exception message added
Actions