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

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: Resource Resolver 1.6.8
• Fix Version/s: Resource Resolver 1.6.12
• Component/s: ResourceResolver
• Assignee: Karl Pauls
• Reporter: Ashok Kumar
• Created: 03/Apr/19 06:17
• Updated: 25/Apr/19 14:05
• Resolved: 09/Apr/19 03:48

Description

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 '/'

Issue Links

• links to: GitHub Pull Request #13