commit 3c537540ac43c7249529b59ce67691d87c241cf2
author Yegor Kozlov <yegor.kozlov@gartner.com> Tue Mar 24 10:40:14 2020 +0300
committer Robert Munteanu <robert.munteanu@gmail.com> Wed Mar 25 14:29:12 2020 +0100
tree 8945a1cc2a65b8d77c33ebc12c0b721c292365fc
parent 78c51f1775f58b87f5d9dca562c4bab3100d0d4c

SLING-9247 - Use Set instead of List to test  ignoreUrlParams

src/main/java/org/apache/sling/dynamicinclude/Configuration.java

diff --git a/src/main/java/org/apache/sling/dynamicinclude/Configuration.java b/src/main/java/org/apache/sling/dynamicinclude/Configuration.java
index ce09163..0fa523a 100755
--- a/src/main/java/org/apache/sling/dynamicinclude/Configuration.java
+++ b/src/main/java/org/apache/sling/dynamicinclude/Configuration.java
@@ -20,6 +20,8 @@
 package org.apache.sling.dynamicinclude;
 
 import java.util.Arrays;
+import java.util.Collection;
+import java.util.HashSet;
 import java.util.List;
 
 import org.apache.commons.lang3.ArrayUtils;
@@ -121,7 +123,7 @@
 
   private boolean disableIgnoreUrlParams;
 
-  private List<String> ignoreUrlParams;
+  private Collection<String> ignoreUrlParams;
 
   private boolean rewritePath;
 
@@ -147,8 +149,9 @@
     addComment = cfg.include$_$filter_config_add__comment();
     includeTypeName = cfg.include$_$filter_config_include$_$type();
     requiredHeader = cfg.include$_$filter_config_required__header();
-    ignoreUrlParams = Arrays.asList(PropertiesUtil.toStringArray(cfg.include$_$filter_config_ignoreUrlParams(),
-        new String[0]));
+    ignoreUrlParams = new HashSet<>(
+            Arrays.asList(PropertiesUtil.toStringArray(cfg.include$_$filter_config_ignoreUrlParams(), new String[0]))
+    );
     rewritePath = cfg.include$_$filter_config_rewrite();
     appendSuffix = cfg.include$_$filter_config_appendSuffix();
     disableIgnoreUrlParams = cfg.include$_$filter_config_disableIgnoreUrlParams();
@@ -219,7 +222,7 @@
     return requiredHeader;
   }
 
-  public List<String> getIgnoreUrlParams() {
+  public Collection<String> getIgnoreUrlParams() {
     return ignoreUrlParams;
   }
 

src/main/java/org/apache/sling/dynamicinclude/IncludeTagFilter.java

diff --git a/src/main/java/org/apache/sling/dynamicinclude/IncludeTagFilter.java b/src/main/java/org/apache/sling/dynamicinclude/IncludeTagFilter.java
index d860549..b43568a 100644
--- a/src/main/java/org/apache/sling/dynamicinclude/IncludeTagFilter.java
+++ b/src/main/java/org/apache/sling/dynamicinclude/IncludeTagFilter.java
@@ -23,6 +23,7 @@
 import java.io.PrintWriter;
 import java.net.URI;
 import java.net.URISyntaxException;
+import java.util.Collection;
 import java.util.Enumeration;
 import java.util.List;
 
@@ -113,7 +114,7 @@
         return StringUtils.isBlank(requiredHeader) || containsHeader(requiredHeader, request);
     }
 
-    private boolean requestHasParameters(List<String> ignoreUrlParams, SlingHttpServletRequest request) {
+    private boolean requestHasParameters(Collection<String> ignoreUrlParams, SlingHttpServletRequest request) {
         final Enumeration<?> paramNames = request.getParameterNames();
         while (paramNames.hasMoreElements()) {
             final String paramName = (String) paramNames.nextElement();