You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ro...@apache.org on 2020/02/18 13:16:53 UTC
[sling-org-apache-sling-dynamic-include] branch master updated:
SLING-9045 Adding option to ignore the processing rules around querystring
params Originally-By: @srgovindaraj
This is an automated email from the ASF dual-hosted git repository.
rombert pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-dynamic-include.git
The following commit(s) were added to refs/heads/master by this push:
new 78c51f1 SLING-9045 Adding option to ignore the processing rules around querystring params Originally-By: @srgovindaraj
78c51f1 is described below
commit 78c51f1775f58b87f5d9dca562c4bab3100d0d4c
Author: John Mitchell <jo...@adobe.com>
AuthorDate: Tue Feb 18 10:01:14 2020 +0000
SLING-9045 Adding option to ignore the processing rules around querystring params
Originally-By: @srgovindaraj
---
README.md | 1 +
.../java/org/apache/sling/dynamicinclude/Configuration.java | 12 +++++++++++-
.../org/apache/sling/dynamicinclude/IncludeTagFilter.java | 3 ++-
3 files changed, 14 insertions(+), 2 deletions(-)
diff --git a/README.md b/README.md
index 85cce96..7fbf083 100644
--- a/README.md
+++ b/README.md
@@ -41,6 +41,7 @@ Filter is delivered as a standard OSGi bundle. SDI is configured via the configu
* **Filter selector** - selector used to get actual content
* **Component TTL** - time to live in seconds, set for rendered component (require Dispatcher 4.1.11+)
* **Required header** - SDI will be enabled only if the configured header is present in the request. By default it's `Server-Agent=Communique-Dispatcher` header, added by the AEM dispatcher. You may enter just the header name only or the name and the value split with `=`.
+* **Disable Ignore URL params check** - SDI will process all requests and discard ignore URL params check including requests with GET params.
* **Ignore URL params** - SDI normally skips all requests containing any GET parameters. This option allows to set a list of parameters that should be ignored in the test. See the [Ignoring URL parameters](https://docs.adobe.com/docs/en/dispatcher/disp-config.html#Ignoring%20URL%20Parameters) section in the dispatcher documentation.
* **Include path rewriting** -- enable rewriting link (according to sling mappings) that is used for dynamic content including.
* **Append suffix** -- ensures that the suffix of the parent request is included with the dynamic include.
diff --git a/src/main/java/org/apache/sling/dynamicinclude/Configuration.java b/src/main/java/org/apache/sling/dynamicinclude/Configuration.java
index 97a33a1..ce09163 100755
--- a/src/main/java/org/apache/sling/dynamicinclude/Configuration.java
+++ b/src/main/java/org/apache/sling/dynamicinclude/Configuration.java
@@ -91,9 +91,12 @@ public class Configuration {
@AttributeDefinition(name = "Include path rewriting", description = "Check to enable include path rewriting")
boolean include$_$filter_config_rewrite() default false;
-
+
@AttributeDefinition(name = "Append suffix to dynamic includes", description = "Check to append the suffix of the parent request to the dynamic include.")
boolean include$_$filter_config_appendSuffix() default true;
+
+ @AttributeDefinition(name = "Disable ignore URL params check", description = "Disable the check in the Ignore URL Params setting.")
+ boolean include$_$filter_config_disableIgnoreUrlParams() default false;
}
private static final Logger LOG = LoggerFactory.getLogger(Configuration.class);
@@ -116,6 +119,8 @@ public class Configuration {
private String requiredHeader;
+ private boolean disableIgnoreUrlParams;
+
private List<String> ignoreUrlParams;
private boolean rewritePath;
@@ -146,6 +151,7 @@ public class Configuration {
new String[0]));
rewritePath = cfg.include$_$filter_config_rewrite();
appendSuffix = cfg.include$_$filter_config_appendSuffix();
+ disableIgnoreUrlParams = cfg.include$_$filter_config_disableIgnoreUrlParams();
}
private PathMatcher choosePathMatcher(String pathPattern) {
@@ -217,6 +223,10 @@ public class Configuration {
return ignoreUrlParams;
}
+ public boolean isDisableIgnoreUrlParams() {
+ return disableIgnoreUrlParams;
+ }
+
public boolean isRewritePath() {
return rewritePath;
}
diff --git a/src/main/java/org/apache/sling/dynamicinclude/IncludeTagFilter.java b/src/main/java/org/apache/sling/dynamicinclude/IncludeTagFilter.java
index 396984b..d860549 100644
--- a/src/main/java/org/apache/sling/dynamicinclude/IncludeTagFilter.java
+++ b/src/main/java/org/apache/sling/dynamicinclude/IncludeTagFilter.java
@@ -105,7 +105,8 @@ public class IncludeTagFilter implements Filter {
}
private boolean shouldWriteIncludes(Configuration config, SlingHttpServletRequest request) {
- if (requestHasParameters(config.getIgnoreUrlParams(), request)) {
+ // Do not skip GET requests when DisableIgnoreUrlParams set to true.
+ if (!config.isDisableIgnoreUrlParams() && requestHasParameters(config.getIgnoreUrlParams(), request)) {
return false;
}
final String requiredHeader = config.getRequiredHeader();