Merge pull request #184 from gbourant/logout

remove cookie correctly
quarkiverse · Jan 3, 2024 · b35a879 · b35a879
2 parents 3d61842 + f010a1b
commit b35a879
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/deployment/src/test/java/io/quarkiverse/renarde/test/JWTTest.java b/deployment/src/test/java/io/quarkiverse/renarde/test/JWTTest.java
@@ -110,7 +110,7 @@ private void assertRedirectWithMessage(String token, String message) {
                 .getValues("Set-Cookie")
                 .stream().filter(c -> c.startsWith("QuarkusUser=")).findFirst().get();
 
-        Assertions.assertEquals("QuarkusUser=;Version=1;Max-Age=0", quarkusUserCookie);
+        Assertions.assertEquals("QuarkusUser=;Version=1;Path=/;Max-Age=0", quarkusUserCookie);
 
         String flash = response.cookie(Flash.FLASH_COOKIE_NAME);
         Map<String, Object> data = Flash.decodeCookieValue(flash);

diff --git a/integration-tests/src/test/java/io/quarkiverse/renarde/it/RenardeOidcTest.java b/integration-tests/src/test/java/io/quarkiverse/renarde/it/RenardeOidcTest.java
@@ -150,7 +150,7 @@ private void verifyLoggedInAndLogout(RenardeCookieFilter cookieFilter, String co
                 .getValues("Set-Cookie")
                 .stream().filter(c -> c.startsWith("QuarkusUser=")).findFirst().get();
 
-        Assertions.assertEquals("QuarkusUser=;Version=1;Max-Age=0", logoutCookie);
+        Assertions.assertEquals("QuarkusUser=;Version=1;Path=/;Max-Age=0", logoutCookie);
 
     }
 

diff --git a/runtime/src/main/java/io/quarkiverse/renarde/util/AuthenticationFailedExceptionMapper.java b/runtime/src/main/java/io/quarkiverse/renarde/util/AuthenticationFailedExceptionMapper.java
@@ -69,7 +69,7 @@ private Response redirectToRoot(String message) {
     }
 
     public static NewCookie invalidateCookie(String cookieName) {
-        return new NewCookie.Builder(cookieName).maxAge(0).build();
+        return new NewCookie.Builder(cookieName).path("/").maxAge(0).build();
     }
 
 }