You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Karl Pauls (JIRA)" <ji...@apache.org> on 2019/04/25 14:06:01 UTC

[jira] [Closed] (SLING-8341) unexpected results from resourceresolver.map when input path has adjacent '/'

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

Karl Pauls closed SLING-8341.
-----------------------------

> unexpected results from resourceresolver.map when input path has adjacent '/'
> -----------------------------------------------------------------------------
>
>                 Key: SLING-8341
>                 URL: https://issues.apache.org/jira/browse/SLING-8341
>             Project: Sling
>          Issue Type: Bug
>          Components: ResourceResolver
>    Affects Versions: Resource Resolver 1.6.8
>            Reporter: Ashok Kumar
>            Assignee: Karl Pauls
>            Priority: Major
>             Fix For: Resource Resolver 1.6.12
>
>         Attachments: Variables-resolveInternal.png
>
>          Time Spent: 50m
>  Remaining Estimate: 0h
>
> Scenario : - ResourceResolver.map(‘/content/we-retail/us*//*en.html’) results in ‘/content/we-retail/us/enn.html’
> Steps to Reproduce :
>  1. Go to console /system/console/jcrresolver
>  2. input link "/content/we-retail/us//en.html" to Configuration Test
>  3. click map
>  4. the result shows
>  Output 
>  Primary: /content/we-retail/us/enn.html. Other candidates: /content/we-retail/us//en.html.
> Analysis:
> Attached is the variable states at Line 772 [https://github.com/apache/sling-org-apache-sling-resourceresolver/blob/master/src/main/java/org/apache/sling/resourceresolver/impl/ResourceResolverImpl.java#L769-L770] 
> [https://github.com/apache/sling-org-apache-sling-resourceresolver/blob/master/src/main/java/org/apache/sling/resourceresolver/impl/ResourceResolverImpl.java#L732-L763] gives resolutionPath = "/content/we-retail/us/en" but original absPath input param had an extra slash so pathinfo gets extra character in this case 'n'. Refer attached variable state. If there are three adjacent slashes , pathinfo would have two extra chars 'en'. 
>  
> Questions: - How to handle such input gracefully 
> what should be the output - throw an exception and exit OR return the original path itself OR automatically remove adjacent slashes and replace that with single '/'



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)