On Thursday 03 April 2014 16:12:47 Antonio Sanso wrote:
> hi *,

hello Antonio,

> SLING-722 introduced an important fix to solve some issue related to sling
> being registered in a Servlet container. Unluckily this fix has been
> removed in SLING-2998.

do you mean the following lines?

-        String pathInfo = request.getPathInfo();
-        if (pathInfo == null || pathInfo.length() == 0) {
-            pathInfo = "/";

Which problem do you see?

> The rationale behind this is not too clear to me. I am not against
> SLING-2998 per se. I think though that SLING-2998 should also contain the
> fix contained in SLING-722.
> 
> @oliver WDYT?

Did you read the thread on [email protected] [2]? I did some research 
back in 2013 and found a blog post [3] with a solution for my problem: "Good 
old Cocoon fixes this by simply concatenating the path info and servlet path, 
so that your application doesn't have to care about it."

This was tested on Sling [4] and CQ and worked well.

> regards
> 
> antonio

hth,
O.

> [0] https://issues.apache.org/jira/browse/SLING-722
> [1] https://issues.apache.org/jira/browse/SLING-2998

[2] http://mail-archives.apache.org/mod_mbox/sling-
users/201308.mbox/<[email protected]>
[3] http://bluxte.net/musings/2006/03/29/servletpath-and-pathinfo-servlet-api-
weirdness
[4] https://github.com/oliverlietz/sling-
tests/tree/master/org.apache.sling.tests/SLING-2998

Reply via email to