Issue #7801 Fix incorrect ConstraintTest for session with form redirect

Signed-off-by: Jan Bartel <janb@webtide.com>
jetty · Mar 30, 2022 · 8122ae6 · 8122ae6
1 parent b86dd9d
commit 8122ae6
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/jetty-security/src/test/java/org/eclipse/jetty/security/ConstraintTest.java b/jetty-security/src/test/java/org/eclipse/jetty/security/ConstraintTest.java
@@ -1013,7 +1013,6 @@ public void testFormRedirect() throws Exception
             "Cookie: JSESSIONID=" + session + "\r\n" +
             "\r\n");
         assertThat(response, startsWith("HTTP/1.1 200 OK"));
-        assertThat(response, containsString("JSESSIONID=" + session));
 
         response = _connector.getResponse("GET /ctx/admin/info HTTP/1.0\r\n" +
             "Cookie: JSESSIONID=" + session + "\r\n" +

diff --git a/jetty-server/src/main/java/org/eclipse/jetty/server/session/AbstractSessionCache.java b/jetty-server/src/main/java/org/eclipse/jetty/server/session/AbstractSessionCache.java
@@ -549,7 +549,7 @@ public void release(String id, Session session) throws Exception
             if ((session.getRequests() <= 0))
             {
                 //reset the idchanged flag
-                session.setIdChanged(false);              
+                session.setIdChanged(false);
 
                 //save the session
                 if (!_sessionDataStore.isPassivating())