Skip to content

Commit

Permalink
[#1844] Remove workaround for jetty/jetty.project#9119
Browse files Browse the repository at this point in the history
  • Loading branch information
grgrzybek committed Mar 23, 2023
1 parent df5b7a1 commit 78e37ca
Showing 1 changed file with 3 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -510,8 +510,7 @@ protected void doGet(HttpServletRequest req, HttpServletResponse resp) throws Se
response = send(port, "/gateway/x?what=forward&where=/");
assertTrue(response.contains("req.context_path=\"\""));
assertTrue(response.contains("req.request_uri=\"/index.y\""));
// see https://github.com/eclipse/jetty.project/issues/9119
assertTrue(response.contains("javax.servlet.forward.context_path=\"/\""));
assertTrue(response.contains("javax.servlet.forward.context_path=\"\""));
assertTrue(response.contains("javax.servlet.forward.request_uri=\"/gateway/x\""));
assertTrue(response.contains("javax.servlet.forward.servlet_path=\"/gateway\""));
assertTrue(response.contains("javax.servlet.forward.path_info=\"/x\""));
Expand Down Expand Up @@ -550,17 +549,15 @@ protected void doGet(HttpServletRequest req, HttpServletResponse resp) throws Se
response = send(port, "/sub/");
assertTrue(response.contains("req.context_path=\"\""));
assertTrue(response.contains("req.request_uri=\"/sub/index.x\""));
// see https://github.com/eclipse/jetty.project/issues/9119
assertTrue(response.contains("javax.servlet.forward.context_path=\"/\""));
assertTrue(response.contains("javax.servlet.forward.context_path=\"\""));
assertTrue(response.contains("javax.servlet.forward.request_uri=\"/sub/\""));
assertTrue(response.contains("javax.servlet.forward.servlet_path=\"/sub/\""));
assertTrue(response.contains("javax.servlet.forward.path_info=\"null\""));

response = send(port, "/gateway/x?what=forward&where=/sub/");
assertTrue(response.contains("req.context_path=\"\""));
assertTrue(response.contains("req.request_uri=\"/sub/index.x\""));
// see https://github.com/eclipse/jetty.project/issues/9119
assertTrue(response.contains("javax.servlet.forward.context_path=\"/\""));
assertTrue(response.contains("javax.servlet.forward.context_path=\"\""));
assertTrue(response.contains("javax.servlet.forward.request_uri=\"/gateway/x\""));
assertTrue(response.contains("javax.servlet.forward.servlet_path=\"/gateway\""));
assertTrue(response.contains("javax.servlet.forward.path_info=\"/x\""));
Expand Down

0 comments on commit 78e37ca

Please sign in to comment.