New logging formats and policies

Author: srnyx

src/main/java/xyz/srnyx/lazylibrary/LazyLibrary.java

@@ -94,7 +94,7 @@ public LazyLibrary() {
         settings.builder.accept(builder);
         builder.build(jda);
 
-        // stop command
+        // Console commands
         new Thread(() -> {
             final Scanner scanner = new Scanner(System.in);
             while (scanner.hasNextLine()) {

src/main/java/xyz/srnyx/lazylibrary/logging/FixedWindowWithTimeRollingPolicy.java

@@ -0,0 +1,24 @@
+package xyz.srnyx.lazylibrary.logging;
+
+import ch.qos.logback.core.rolling.FixedWindowRollingPolicy;
+
+import org.jetbrains.annotations.NotNull;
+
+import java.time.OffsetDateTime;
+import java.time.ZoneId;
+import java.time.format.DateTimeFormatter;
+import java.util.regex.Pattern;
+
+
+// https://stackoverflow.com/a/72377192/18122334
+public class FixedWindowWithTimeRollingPolicy extends FixedWindowRollingPolicy {
+    @NotNull private static final ZoneId ZONE_ID = ZoneId.of("America/New_York");
+    @NotNull private static final Pattern DATE_PATTERN = Pattern.compile("%d\\{([^}]+)}");
+
+    @Override
+    public void setFileNamePattern(String fnp) {
+        // Replace %d{FORMAT} with DateTimeFormatter.ofPattern(FORMAT)
+        final OffsetDateTime now = OffsetDateTime.now(ZONE_ID);
+        super.setFileNamePattern(DATE_PATTERN.matcher(fnp).replaceAll(m -> DateTimeFormatter.ofPattern(m.group(1)).format(now)));
+    }
+}

src/main/java/xyz/srnyx/lazylibrary/logging/StartupBasedTriggeringPolicy.java

@@ -0,0 +1,21 @@
+package xyz.srnyx.lazylibrary.logging;
+
+import ch.qos.logback.core.rolling.TriggeringPolicyBase;
+
+import java.io.File;
+
+
+// https://stackoverflow.com/a/32203808/18122334
+public class StartupBasedTriggeringPolicy<T> extends TriggeringPolicyBase<T> {
+    private boolean doRolling = true;
+
+    @Override
+    public boolean isTriggeringEvent(File activeFile, T event) {
+        // Roll first time when event gets called
+        if (doRolling) {
+            doRolling = false;
+            return true;
+        }
+        return false;
+    }
+}

src/main/resources/logback.xml

@@ -1,28 +1,25 @@
-<?xml version="1.0" encoding="UTF-8"?>
 <configuration>
-    <timestamp key="bySecond" datePattern="yyyyMMdd'T'HHmmss"/>
+    <property name="TIMESTAMP" value="%d{hh:mm:ss.SSS a}"/>
+    <property name="LOGS_PATH" value="logs/"/>
 
     <appender name="STDOUT" class="ch.qos.logback.core.ConsoleAppender">
         <encoder>
-            <!-- Debug pattern -->
-            <!--<pattern>%d{HH:mm:ss.SSS} %boldCyan(%-26.-26thread) %boldRed(%-36.-36class{36}) %boldRed(#%-24.-24method{24}) %boldRed(L%-5.-5line) %boldYellow(%-20.-20logger{0}) %highlight(%-6level) %msg%n%throwable</pattern>-->
-            <!-- Normal pattern, no stack frames -->
-            <pattern>%d{HH:mm:ss.SSS} %highlight(%-6level) %boldCyan(%-20.-20thread) %boldYellow(%-20.-20logger{0}) %msg%n%throwable</pattern>
+            <pattern>${TIMESTAMP} %highlight(%-6level) %boldYellow(%-15.-15logger{0}) %boldCyan(%-30.-30class{30}) %msg%n%throwable</pattern>
         </encoder>
     </appender>
 
     <appender name="FILE" class="ch.qos.logback.core.rolling.RollingFileAppender">
-        <rollingPolicy class="ch.qos.logback.core.rolling.TimeBasedRollingPolicy">
-            <!-- daily rollover -->
-            <fileNamePattern>logs/logs-%d{yyyy-MM-dd}.log</fileNamePattern>
+        <file>${LOG_PATH}bot.log</file>
 
-            <!-- keep 90 days' worth of history capped at 3GB total size -->
-            <maxHistory>90</maxHistory>
-            <totalSizeCap>3GB</totalSizeCap>
+        <rollingPolicy class="xyz.srnyx.lazylibrary.logging.FixedWindowWithTimeRollingPolicy">
+            <fileNamePattern>${LOG_PATH}bot_%d{yyyy-MM-dd_HH-mm}_%i.log</fileNamePattern>
+            <maxIndex>10</maxIndex>
         </rollingPolicy>
 
+        <triggeringPolicy class="xyz.srnyx.lazylibrary.logging.StartupBasedTriggeringPolicy"/>
+
         <encoder>
-            <pattern>%d{HH:mm:ss.SSS} %-26.-26thread %-36.-36class{36} #%-24.-24method{24} L%-5.-5line %-20.-20logger{0} %-6level %msg%n%throwable</pattern>
+            <pattern>${TIMESTAMP} [%-6level] [%-15.-15logger{0}] [%-30.-30class{30}] %msg%n%throwable</pattern>
         </encoder>
     </appender>