Skip to content

Commit 73adc59

Browse files
Jason Wattlukeis
authored andcommitted
Treat http status 400 and 401 as incomplete new requests (#2825)
1 parent 3aff378 commit 73adc59

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

java/server/src/org/openqa/grid/internal/TestSession.java

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -234,10 +234,15 @@ public String forward(SeleniumBasedRequest request, HttpServletResponse response
234234

235235
byte[] consumedNewWebDriverSessionBody = null;
236236
if (statusCode != HttpServletResponse.SC_INTERNAL_SERVER_ERROR &&
237-
statusCode != HttpServletResponse.SC_NOT_FOUND) {
237+
statusCode != HttpServletResponse.SC_NOT_FOUND &&
238+
statusCode != HttpServletResponse.SC_BAD_REQUEST &&
239+
statusCode != HttpServletResponse.SC_UNAUTHORIZED) {
238240
consumedNewWebDriverSessionBody = updateHubIfNewWebDriverSession(request, proxyResponse);
239241
}
240-
if (newSessionRequest && statusCode == HttpServletResponse.SC_INTERNAL_SERVER_ERROR) {
242+
if (newSessionRequest &&
243+
(statusCode == HttpServletResponse.SC_INTERNAL_SERVER_ERROR ||
244+
statusCode == HttpServletResponse.SC_BAD_REQUEST ||
245+
statusCode == HttpServletResponse.SC_UNAUTHORIZED)) {
241246
removeIncompleteNewSessionRequest();
242247
}
243248
if (statusCode == HttpServletResponse.SC_NOT_FOUND) {

0 commit comments

Comments
 (0)