[GitHub] [sling-org-apache-sling-engine] kwin commented on a diff in pull request #23: SLING-11438 - Resource path consising of %7D with multiple dots leads…

[GitBox <gi...@apache.org>]

kwin commented on code in PR #23:
URL: https://github.com/apache/sling-org-apache-sling-engine/pull/23#discussion_r914737508


##########
src/main/java/org/apache/sling/engine/impl/request/RequestData.java:
##########
@@ -763,4 +768,34 @@ public SlingHttpServletRequest createRequest(RequestData requestData, HttpServle
     private static interface SlingHttpServletRequestFactory {
         SlingHttpServletRequest createRequest(RequestData requestData, HttpServletRequest request);
     }
+
+    /*
+    * Traverses the path segment wise and checks
+    * if there is any path with only dots (".")
+    * skipping SKIPPED_TRAVERSAL_CHARS characters in segment.
+    */
+    private static boolean travesesParentPath(String path) {

Review Comment:
   small typo here: traversal



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@sling.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org