forrest-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From David Crossley <cross...@apache.org>
Subject Re: [jira] Commented: (FOR-980) RSS - Current_changes rss shows the whole changes.rss
Date Tue, 26 Jun 2007 01:56:33 GMT
Gavin (JIRA) wrote:
> 
>     [ https://issues.apache.org/jira/browse/FOR-980?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12507847
] 
> 
> Gavin commented on FOR-980:
> ---------------------------
> 
> Additionally there is a coding mistake somewhere, as you found out, putting current_changes.*
instead of changes_current.* will produce a complete list as if it were just changes.*

Because the sitemap pattern is "^(.*?)([^/]*)changes_*.xml"
i.e. anything at the start, followed by anything that is
not a slash, followed by "changes_".

We could tighten the pattern with a "/" before "changes"
but then we need to specifically catch the root case
localhost:8888/changes...

I suggest not touching it. People should use the URIs
as specified in the projectInfo plugin docs.

These are examples of situations were we really need
to use "Content Aware Pipelines" rather than pattern
matching for the outer match. Leave that for another day.

> I also tried these :-
> 
> chan_current
> c_current
> _current
> 234efwerwerwerr233rr23_current
> 
> all of these produce the same complete changes list.

Really? They all fail for me.

-David

Mime
View raw message