ambidextrous/gemp-lotr
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

upgrading http traces to infos, attempting to expand logging in general

Browse Source
This commit is contained in:
Christian 'ketura' McCarty
2022-11-02 23:18:26 -05:00
parent a9348b1a37
commit caccf5f970
2 changed files with 30 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
gemp-lotr/gemp-lotr-async/src/main/java/com/gempukku/lotro/async/GempukkuHttpRequestHandler.java
View File

@@ -97,7 +97,7 @@ public class GempukkuHttpRequestHandler extends SimpleChannelInboundHandler<Full
try { try {
if (isBanned(requestInformation.remoteIp)) { if (isBanned(requestInformation.remoteIp)) {
responseSender.writeError(401); 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 { else {
_uriRequestHandler.handleRequest(uri, httpRequest, _objects, responseSender, requestInformation.remoteIp); _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, //401, 403, 404, and other 400 errors should just do minimal logging,
// but 400 itself should error out // but 400 itself should error out
if(code % 400 < 100 && code != 400) { 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 // record an HTTP 400
else if(code == 400 || code % 500 < 100) { else if(code == 400 || code % 500 < 100) {

28
gemp-lotr/gemp-lotr-async/src/main/resources/log4j.xml
View File

@@ -26,18 +26,46 @@
</layout> </layout>
</appender> </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"> <logger name="com.gempukku" additivity="false">
<level value="debug"/> <level value="debug"/>
<appender-ref ref="R"/> <appender-ref ref="R"/>
<appender-ref ref="stdout"/> <appender-ref ref="stdout"/>
<appender-ref ref="tracelog"/>
</logger> </logger>
<logger name="chat"> <logger name="chat">
<level value="ALL"/> <level value="ALL"/>
<appender-ref ref="chat"/> <appender-ref ref="chat"/>
<appender-ref ref="tracelog"/>
</logger> </logger>
<root> <root>
<level value="error"/> <level value="error"/>
<appender-ref ref="R"/> <appender-ref ref="R"/>
<appender-ref ref="stdout"/> <appender-ref ref="stdout"/>
<appender-ref ref="tracelog"/>
<appender-ref ref="errorlog"/>
</root>
<root>
<level value="trace"/>
<appender-ref ref="tracelog"/>
</root> </root>
</log4j:configuration> </log4j:configuration>