FELIX-4925 : Request path is not decoded
git-svn-id: https://svn.apache.org/repos/asf/felix/trunk@1685217 13f79535-47bb-0310-9956-ffa450edef68
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 4c375f9..ad7a327 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
@@ -589,12 +589,6 @@
{
path = path.concat(req.getPathInfo());
}
- // Workaround to get path parameters (FELIX-4925)
- // This fails if the path part contains encoded characters!
- if ( req.getRequestURI().contains(";") )
- {
- path = UriUtils.relativePath(req.getContextPath(), req.getRequestURI());
- }
final String requestURI = path;
// Determine which servlet we should forward the request to...
diff --git a/http/itest/src/test/java/org/apache/felix/http/itest/HttpJettyTest.java b/http/itest/src/test/java/org/apache/felix/http/itest/HttpJettyTest.java
index b423c6f..31f91c7 100644
--- a/http/itest/src/test/java/org/apache/felix/http/itest/HttpJettyTest.java
+++ b/http/itest/src/test/java/org/apache/felix/http/itest/HttpJettyTest.java
@@ -329,45 +329,6 @@
}
/**
- * Tests the handling of URI path parameters.
- */
- @Test
- public void testHandleUriPathParametersOk() throws Exception
- {
- CountDownLatch initLatch = new CountDownLatch(1);
- CountDownLatch destroyLatch = new CountDownLatch(1);
-
- TestServlet servlet = new TestServlet(initLatch, destroyLatch)
- {
- private static final long serialVersionUID = 1L;
-
- @Override
- protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException
- {
- assertEquals("", request.getContextPath());
- assertEquals("/foo", request.getServletPath());
- assertEquals("/a;b/c;d/e;f;g/h", request.getPathInfo());
- assertEquals("/foo/a;b/c;d/e;f;g/h", request.getRequestURI());
- assertEquals("i=j+k&l=m", request.getQueryString());
- }
- };
-
- register("/foo", servlet);
-
- URL testURL = createURL("/foo/a;b/c;d/e;f;g/h?i=j+k&l=m");
-
- assertTrue(initLatch.await(5, TimeUnit.SECONDS));
-
- assertResponseCode(SC_OK, testURL);
-
- unregister(servlet);
-
- assertTrue(destroyLatch.await(5, TimeUnit.SECONDS));
-
- assertResponseCode(SC_NOT_FOUND, testURL);
- }
-
- /**
* Tests that we can register a filter with Jetty and that its lifecycle is correctly controlled.
*/
@Test