Skip to content

Commit 48c16de

Browse files
committed
Issue #8578 - test both request URL/URI results
Signed-off-by: Joakim Erdfelt <[email protected]>
1 parent d3c7ee3 commit 48c16de

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

jetty-server/src/test/java/org/eclipse/jetty/server/RequestTest.java

+6-3
Original file line numberDiff line numberDiff line change
@@ -876,10 +876,12 @@ public boolean check(HttpServletRequest request, HttpServletResponse response)
876876
@Test
877877
public void testConnectRequestURL() throws Exception
878878
{
879-
final AtomicReference<String> result = new AtomicReference<>();
879+
final AtomicReference<String> resultRequestURL = new AtomicReference<>();
880+
final AtomicReference<String> resultRequestURI = new AtomicReference<>();
880881
_handler._checker = (request, response) ->
881882
{
882-
result.set("" + request.getRequestURL());
883+
resultRequestURL.set("" + request.getRequestURL());
884+
resultRequestURI.set("" + request.getRequestURI());
883885
return true;
884886
};
885887

@@ -890,7 +892,8 @@ public void testConnectRequestURL() throws Exception
890892
"\n");
891893
HttpTester.Response response = HttpTester.parseResponse(rawResponse);
892894
assertThat(response.getStatus(), is(HttpStatus.OK_200));
893-
assertThat(result.get(), is("http://myhost:9999"));
895+
assertThat(resultRequestURL.get(), is("http://myhost:9999"));
896+
assertThat(resultRequestURI.get(), is("myhost:9999"));
894897
}
895898

896899
@Test

0 commit comments

Comments
 (0)