Changes done to improve error log percentage

Issue-ID: PORTAL-1060
Signed-off-by: Kishore Reddy Dwaram <kishore.reddy.dwaram@att.com>
Change-Id: I68da1e117c71ccfd88251d868827ae571ada9349

diff --git a/ecomp-portal-BE-os/src/main/java/org/onap/portalapp/filter/SecurityXssValidator.java b/ecomp-portal-BE-os/src/main/java/org/onap/portalapp/filter/SecurityXssValidator.java
index 3adc313..cc4ba85 100644 (file)
--- a/ecomp-portal-BE-os/src/main/java/org/onap/portalapp/filter/SecurityXssValidator.java
+++ b/ecomp-portal-BE-os/src/main/java/org/onap/portalapp/filter/SecurityXssValidator.java
@@ -160,18 +160,21 @@ public class SecurityXssValidator {
                                        logger.info(EELFLoggerDelegate.applicationLogger, "denyXSS() replacing &timeseclgn with empty string for request value : " + value);
                                        value=value.replaceAll("&timeseclgn", "");
                                }
+                               while(value.contains("%25")) {
+                                       value = value.replaceAll("%25", "%");
+                               }
                                value = ESAPI.encoder().canonicalize(value);
                                for (Pattern xssInputPattern : XSS_INPUT_PATTERNS) {
                                        if (xssInputPattern.matcher(value).matches()) {
                                                flag = Boolean.TRUE;
                                                break;
                                        }
-
                                }
                        }
 
                } catch (Exception e) {
-                       logger.error(EELFLoggerDelegate.errorLogger, "denyXSS() failed for request with value : " + value, e);
+                       logger.error(EELFLoggerDelegate.errorLogger, "denyXSS() failed for request with value : " + e.getMessage());
+                       logger.debug(EELFLoggerDelegate.debugLogger, "denyXSS() failed for request with value : " + value, e);
                }
 
                return flag;
@@ -208,6 +211,5 @@ public class SecurityXssValidator {
        public void setXSS_INPUT_PATTERNS(List<Pattern> xSS_INPUT_PATTERNS) {
                XSS_INPUT_PATTERNS = xSS_INPUT_PATTERNS;
        }
-       
 
 }
\ No newline at end of file