[ https://issues.apache.org/jira/browse/SLING-8341?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16810605#comment-16810605 ]
Ashok Kumar commented on SLING-8341: ------------------------------------ [~karlpauls], [~cziegeler] Please review the PR - [https://github.com/apache/sling-org-apache-sling-resourceresolver/pull/13] Took approach of adjusting the extra slash counts while evaluating the pathinfo. Please suggest if there is a better alternative. Thank you. > 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.10 > > Attachments: Variables-resolveInternal.png > > Time Spent: 10m > 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)