You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Maciej Dybek (JIRA)" <ji...@apache.org> on 2015/05/14 14:17:01 UTC

[jira] [Updated] (SLING-4715) Event filter in ResourceEventDistributionTrigger is wrongly configured

     [ https://issues.apache.org/jira/browse/SLING-4715?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Maciej Dybek updated SLING-4715:
--------------------------------
    Description: 
In line 88 and 89 of [ResourceEventDistributionTrigger|https://github.com/apache/sling/blob/trunk/contrib/extensions/distribution/core/src/main/java/org/apache/sling/distribution/trigger/impl/ResourceEventDistributionTrigger.java] class we have:
{code}
properties.put(EventConstants.EVENT_FILTER, "(path=" + path + "/*)");
properties.put(EventConstants.EVENT_FILTER, "(!(" + DEAConstants.PROPERTY_APPLICATION + "=*))");
{code}

That way the event filter for a specifi path (line 88) is overriden by a filter from line 99 (distributed events). Hence trigger is distributing too many distribution requests. They of course are skipped by allowedRoots property in SimpleDistributionAgent but nevertheless it's an issue.

  was:
In line 88 and 89 of [ResourceEventDistributionTrigger|https://github.com/apache/sling/blob/trunk/contrib/extensions/distribution/core/src/main/java/org/apache/sling/distribution/trigger/impl/ResourceEventDistributionTrigger.java] class we have:
{code}
properties.put(EventConstants.EVENT_FILTER, "(path=" + path + "/*)");
properties.put(EventConstants.EVENT_FILTER, "(!(" + DEAConstants.PROPERTY_APPLICATION + "=*))");
{code}

That way the event filter for a specifi path (line 88) is overriden by a filter from line 99. Hence trigger is distributing too many distribution requests. They of course are skipped by allowedRoots property in SimpleDistributionAgent but nevertheless it's an issue.


> Event filter in ResourceEventDistributionTrigger is wrongly configured
> ----------------------------------------------------------------------
>
>                 Key: SLING-4715
>                 URL: https://issues.apache.org/jira/browse/SLING-4715
>             Project: Sling
>          Issue Type: Bug
>          Components: Distribution
>    Affects Versions: Content Distribution Core 0.1.1
>            Reporter: Maciej Dybek
>
> In line 88 and 89 of [ResourceEventDistributionTrigger|https://github.com/apache/sling/blob/trunk/contrib/extensions/distribution/core/src/main/java/org/apache/sling/distribution/trigger/impl/ResourceEventDistributionTrigger.java] class we have:
> {code}
> properties.put(EventConstants.EVENT_FILTER, "(path=" + path + "/*)");
> properties.put(EventConstants.EVENT_FILTER, "(!(" + DEAConstants.PROPERTY_APPLICATION + "=*))");
> {code}
> That way the event filter for a specifi path (line 88) is overriden by a filter from line 99 (distributed events). Hence trigger is distributing too many distribution requests. They of course are skipped by allowedRoots property in SimpleDistributionAgent but nevertheless it's an issue.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)