Update the logging properties to opt-out of the prefix events, third …

…iteration
mickeyz07 · Jul 1, 2024 · 6d8f307 · 6d8f307
1 parent d3f2a2f
commit 6d8f307
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/src/main/java/org/owasp/esapi/logging/appender/LogPrefixAppender.java b/src/main/java/org/owasp/esapi/logging/appender/LogPrefixAppender.java
@@ -46,14 +46,11 @@ public class LogPrefixAppender implements LogAppender {
      * @param logServerIp        Whether or not to record server ip information
      * @param logApplicationName Whether or not to record application name
      * @param appName            Application Name to record.
+     * @param logPrefix          is set by default to true
      */
+    @SuppressWarnings("JavadocReference")
     public LogPrefixAppender(boolean logUserInfo, boolean logClientInfo, boolean logServerIp, boolean logApplicationName, String appName) {
-        this.logUserInfo = logUserInfo;
-        this.logClientInfo = logClientInfo;
-        this.logServerIp = logServerIp;
-        this.logApplicationName = logApplicationName;
-        this.appName = appName;
-        this.logPrefix = true;
+        this(logUserInfo, logClientInfo, logServerIp, logApplicationName, appName, true);
     }
 
     /**

diff --git a/src/test/java/org/owasp/esapi/logging/appender/LogPrefixAppenderTest.java b/src/test/java/org/owasp/esapi/logging/appender/LogPrefixAppenderTest.java
@@ -34,7 +34,6 @@ public class LogPrefixAppenderTest {
     private String testLogMessage =  testName.getMethodName() + "-MESSAGE";
     private String testApplicationName = testName.getMethodName() + "-APPLICATION_NAME";
     private EventType testEventType = Logger.EVENT_UNSPECIFIED;
-    //private boolean testIgnorePrefix = true;
 
     private EventTypeLogSupplier etlsSpy;
     private ClientInfoSupplier cisSpy;