[ 
https://issues.apache.org/jira/browse/JCRVLT-500?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17287066#comment-17287066
 ] 

Timothee Maret commented on JCRVLT-500:
---------------------------------------

[~kwin] I just found out that the PR 
[#121|https://github.com/apache/jackrabbit-filevault/pull/121] contained the 
workaround for tests from 
[5e7ac66da|https://github.com/tmaret/jackrabbit-filevault/commit/5e7ac66da9c92f4e8150e3a57c391e543e561474].

The PR [#123|https://github.com/apache/jackrabbit-filevault/pull/123] removes 
the workaround from the test and will increase the test coverage of the fix.

> DefaultPathFilter#isAbsolute fails with quoted patterns
> -------------------------------------------------------
>
>                 Key: JCRVLT-500
>                 URL: https://issues.apache.org/jira/browse/JCRVLT-500
>             Project: Jackrabbit FileVault
>          Issue Type: Bug
>          Components: Packaging
>    Affects Versions: 3.4.8
>            Reporter: Timothee Maret
>            Priority: Major
>             Fix For: 3.4.10
>
>
> DefaultPathFilter#isAbsolute and DefaultPathFilter#translate decide if the 
> pattern corresponds to an absolute path with
> {code}
> pattern.startsWith("/")
> {code}
> This implementation fails when passing a quoted pattern as noted in 
> SLING-10088.
> Indeed, the path {{/some/great/content}} gets quoted as 
> {{\Q/some/great/content\E}} which does not start with a {{/}} although the 
> pattern corresponds to an absolute path.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to