From caccf5f9702c19dd6d812edba03ca37394f5929e Mon Sep 17 00:00:00 2001
From: Christian 'ketura' McCarty <hyperlink.teltura@gmail.com>
Date: Wed, 2 Nov 2022 23:18:26 -0500
Subject: [PATCH] upgrading http traces to infos, attempting to expand logging
 in general

---
 .../async/GempukkuHttpRequestHandler.java     |  4 +--
 .../src/main/resources/log4j.xml              | 28 +++++++++++++++++++
 2 files changed, 30 insertions(+), 2 deletions(-)

diff --git a/gemp-lotr/gemp-lotr-async/src/main/java/com/gempukku/lotro/async/GempukkuHttpRequestHandler.java b/gemp-lotr/gemp-lotr-async/src/main/java/com/gempukku/lotro/async/GempukkuHttpRequestHandler.java
index e8c54d5ab..249334f35 100644
--- a/gemp-lotr/gemp-lotr-async/src/main/java/com/gempukku/lotro/async/GempukkuHttpRequestHandler.java
+++ b/gemp-lotr/gemp-lotr-async/src/main/java/com/gempukku/lotro/async/GempukkuHttpRequestHandler.java
@@ -97,7 +97,7 @@ public class GempukkuHttpRequestHandler extends SimpleChannelInboundHandler<Full
         try {
             if (isBanned(requestInformation.remoteIp)) {
                 responseSender.writeError(401);
-                _log.trace("Denying entry to user from banned IP " + requestInformation.remoteIp);
+                _log.info("Denying entry to user from banned IP " + requestInformation.remoteIp);
             }
             else {
                 _uriRequestHandler.handleRequest(uri, httpRequest, _objects, responseSender, requestInformation.remoteIp);
@@ -107,7 +107,7 @@ public class GempukkuHttpRequestHandler extends SimpleChannelInboundHandler<Full
             //401, 403, 404, and other 400 errors should just do minimal logging,
             // but 400 itself should error out
             if(code % 400 < 100 && code != 400) {
-                _log.trace("HTTP " + code + " response for " + requestInformation.remoteIp + ": " + requestInformation.uri);
+                _log.info("HTTP " + code + " response for " + requestInformation.remoteIp + ": " + requestInformation.uri);
             }
             // record an HTTP 400
             else if(code == 400 || code % 500 < 100) {
diff --git a/gemp-lotr/gemp-lotr-async/src/main/resources/log4j.xml b/gemp-lotr/gemp-lotr-async/src/main/resources/log4j.xml
index 644c5a647..4e43bdf34 100644
--- a/gemp-lotr/gemp-lotr-async/src/main/resources/log4j.xml
+++ b/gemp-lotr/gemp-lotr-async/src/main/resources/log4j.xml
@@ -26,18 +26,46 @@
         </layout>
     </appender>
 
+    <appender name="tracelog" class="org.apache.log4j.RollingFileAppender">
+        <param name="file" value="logs/trace.log"/>
+        <param name="MaxFileSize" value="5MB"/>
+        <!-- Keep one backup file -->
+        <param name="MaxBackupIndex" value="1"/>
+        <layout class="org.apache.log4j.PatternLayout">
+            <param name="ConversionPattern" value="%d{yyyy-MM-dd HH:mm:ss,SSS} %5p %c{1}:%L - %m%n"/>
+        </layout>
+    </appender>
+
+    <appender name="errorlog" class="org.apache.log4j.RollingFileAppender">
+        <param name="file" value="logs/error.log"/>
+        <param name="MaxFileSize" value="5MB"/>
+        <!-- Keep one backup file -->
+        <param name="MaxBackupIndex" value="1"/>
+        <layout class="org.apache.log4j.PatternLayout">
+            <param name="ConversionPattern" value="%d{yyyy-MM-dd HH:mm:ss,SSS} %5p %c{1}:%L - %m%n"/>
+        </layout>
+    </appender>
+
     <logger name="com.gempukku" additivity="false">
         <level value="debug"/>
         <appender-ref ref="R"/>
         <appender-ref ref="stdout"/>
+        <appender-ref ref="tracelog"/>
     </logger>
     <logger name="chat">
         <level value="ALL"/>
         <appender-ref ref="chat"/>
+        <appender-ref ref="tracelog"/>
     </logger>
     <root>
         <level value="error"/>
         <appender-ref ref="R"/>
         <appender-ref ref="stdout"/>
+        <appender-ref ref="tracelog"/>
+        <appender-ref ref="errorlog"/>
+    </root>
+    <root>
+        <level value="trace"/>
+        <appender-ref ref="tracelog"/>
     </root>
 </log4j:configuration>