|
Wojciech Kapcia (Tigase) opened 4 years ago
|
||||||
|
Minor, "cosmetic" change, let's push it to 8.3.0 |
||||||
|
I'm not sure if we shouldn't go slf4j (MIT licence simple facade, I'd say "standard" in projects, which offers more flexibility and we already do have it as dependency because some dependencies already required it) instead of quite limited System.Logger, which seems to be intended more for internal use even, as per https://openjdk.org/jeps/264:
|
||||||
|
Consider using MDC (https://logback.qos.ch/manual/mdc.html; for JID, sesssion-id, connection-id) |
||||||
|
Wojciech Kapcia (Tigase) batch edited 10 months ago
|
||||||
|
Wojciech Kapcia (Tigase) added "Related" #1034 10 months ago
|
||||||
|
Wojciech Kapcia (Tigase) changed fields 10 months ago
|
||||||
|
Wojciech Kapcia (Tigase) added to iteration "tigase-server-9.0.0" 10 months ago
|
||||||
|
Wojciech Kapcia (Tigase) removed from iteration "tigase-server-8.4.0" 10 months ago
|
||||||
|
In the end I think we should go with I started exploring OpenRewrite (https://docs.openrewrite.org/) to handle the migration though recipes to migrate away from JUL are either non-existent or utterly rudimentary. |
||||||
|
Wojciech Kapcia (Tigase) changed title 9 months ago
|
||||||
|
Wojciech Kapcia (Tigase) referenced from other issue 8 months ago
|
||||||
Referenced from commit 7 months ago
|
|||||||
Referenced from commit 7 months ago
|
|||||||
|
|
Type |
Task
|
Priority |
Minor
|
Assignee | |
Version |
tigase-server-9.0.0
|
-
tigase-server-9.0.0 Open
In Java9 JEP 264: Platform Logging API and Service was implemented. It introduces a minimal logging API (which defaults to JUL, but makes it possible to use alternative logging solutions easier). Considering that with Tigase 8.2 we bumped minimal Java version to 11 we should update our sources to use System.Logger.
Ref: Java 9 Platform Logging API
if (Logger.isLoggable) {}
and substitution with lazily evaluated lambda calls (though, they don't accept parameter list so it would probably require string concatenation orString::formatted
…)Items: