Update the logging properties to opt-out of the prefix events ESAPI#844…

… fourt iteration
mickeyz07 · Jul 1, 2024 · 7822d6f · 7822d6f
1 parent 6d8f307
commit 7822d6f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 9 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
@@ -39,7 +39,7 @@ public class LogPrefixAppender implements LogAppender {
     private final boolean logPrefix;
 
     /**
-     * Ctr.
+     * Constructor
      *
      * @param logUserInfo      Whether or not to record user information
      * @param logClientInfo      Whether or not to record client information
@@ -54,7 +54,7 @@ public LogPrefixAppender(boolean logUserInfo, boolean logClientInfo, boolean log
     }
 
     /**
-     * Ctr.
+     * Constructor
      *
      * @param logUserInfo        Whether or not to record user information
      * @param logClientInfo      Whether or not to record client information

diff --git a/src/main/java/org/owasp/esapi/logging/appender/ServerInfoSupplier.java b/src/main/java/org/owasp/esapi/logging/appender/ServerInfoSupplier.java
@@ -58,14 +58,9 @@ public String get() {
                 appInfo.append(request.getLocalAddr()).append(":").append(request.getLocalPort());
             }
         }
-
+        
         if (this.logAppName) {
-            if (this.applicationName != null && !this.applicationName.isEmpty()) {
-                appInfo.append("/").append(this.applicationName);
-            }
-            else if (this.applicationName == null) {
-                appInfo.append("/").append(this.applicationName);
-            }
+            appInfo.append("/").append(this.applicationName);
         }
 
         if (this.logLogName) {