commit 0474147436561858bdb6844e0ccb8f3d0c9f7550
author Carsten Ziegeler <cziegeler@apache.org> Fri Feb 20 16:09:42 2015 +0000
committer Carsten Ziegeler <cziegeler@apache.org> Fri Feb 20 16:09:42 2015 +0000
tree defdf329d08272fc509aa1b9925f1f2ba8f51ba0
parent 1c5065e6534474f853d72e302c22947795596e79

FELIX-4060 : Fix context path handling if container is not deployed in the root context

git-svn-id: https://svn.apache.org/repos/asf/felix/trunk@1661148 13f79535-47bb-0310-9956-ffa450edef68

http/base/src/main/java/org/apache/felix/http/base/internal/dispatch/Dispatcher.java

diff --git a/http/base/src/main/java/org/apache/felix/http/base/internal/dispatch/Dispatcher.java b/http/base/src/main/java/org/apache/felix/http/base/internal/dispatch/Dispatcher.java
index 416069e..906426d 100644
--- a/http/base/src/main/java/org/apache/felix/http/base/internal/dispatch/Dispatcher.java
+++ b/http/base/src/main/java/org/apache/felix/http/base/internal/dispatch/Dispatcher.java
@@ -339,12 +339,11 @@
         @Override
         public String getPathInfo()
         {
-            String pathInfo = super.getPathInfo();
-            if (isForwardingDispatcher() || !isWrapperFor(ServletRequestWrapper.class))
+            if ( this.isInclusionDispatcher() )
             {
-                pathInfo = this.requestInfo.pathInfo;
+                return super.getPathInfo();
             }
-            return pathInfo;
+            return this.requestInfo.pathInfo;
         }
 
         @Override
@@ -391,12 +390,11 @@
         @Override
         public String getRequestURI()
         {
-            String requestURI = super.getRequestURI();
-            if (isForwardingDispatcher() || !isWrapperFor(ServletRequestWrapper.class))
+            if ( isInclusionDispatcher() )
             {
-                requestURI = concat(getContextPath(), this.requestInfo.requestURI);
+                return super.getRequestURI();
             }
-            return requestURI;
+            return concat(getContextPath(), this.requestInfo.requestURI);
         }
 
         @Override
@@ -408,16 +406,11 @@
         @Override
         public String getServletPath()
         {
-            String servletPath = super.getServletPath();
-            if (isForwardingDispatcher() || !isWrapperFor(ServletRequestWrapper.class))
+            if ( isInclusionDispatcher() )
             {
-                servletPath = this.requestInfo.servletPath;
+                return super.getServletPath();
             }
-            if ("/".equals(servletPath))
-            {
-                return ""; // XXX still necessary?
-            }
-            return servletPath;
+            return this.requestInfo.servletPath;
         }
 
         @Override
@@ -584,14 +577,14 @@
         }
 
         // strip of context path
-        requestURI = requestURI.substring(servletHandler.getContext().getContextPath().length());
+        requestURI = requestURI.substring(servletHandler.getContext().getContextPath().length() - req.getContextPath().length());
 
         final String servletPath = servletHandler.determineServletPath(requestURI);
         String pathInfo = UriUtils.compactPath(UriUtils.relativePath(servletPath, requestURI));
         String queryString = null; // XXX
 
         ExtServletContext servletContext = (servletHandler != null) ? servletHandler.getContext() : null;
-        RequestInfo requestInfo = new RequestInfo(servletPath, pathInfo, queryString);
+        final RequestInfo requestInfo = new RequestInfo(servletPath, pathInfo, queryString);
 
         final HttpServletRequest wrappedRequest = new ServletRequestWrapper(req, servletContext, requestInfo, servletHandler.getContextServiceId());
         final FilterHandler[] filterHandlers = this.handlerRegistry.getFilterHandlers(servletHandler, req.getDispatcherType(), requestURI);