[SLING-639] Bug in SimpleWebDAVServlet

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: JCR Webdav 2.0.2
• Fix Version/s: JCR Webdav 2.0.6
• Component/s: JCR
• Assignee: Felix Meschberger
• Reporter: Felix Meschberger
• Created: 02/Sep/08 20:41
• Updated: 03/Sep/08 06:54
• Resolved: 02/Sep/08 20:47

Description

Reported by Marc Speck in [1]

When PathInfo is null or "/", a response is already created and it makes no
sense to call super.service() anymore:

Index: SimpleWebDavServlet.java
===================================================================
— SimpleWebDavServlet.java (revision 691120)
+++ SimpleWebDavServlet.java (working copy)
@@ -85,9 +85,9 @@
uri += slingRepo.getDefaultWorkspace();
response.sendRedirect(uri);
}
+ } else

{ + super.service(request, response); }

-

• super.service(request, response);
  }

// ---------- SCR integration
----------------------------------------------

Regards,
Marc

[1] http://markmail.org/message/z3tp43u54a7yqhqn