Skip to content

Commit 9dc2517

Browse files
committed
fix: removing cookies for specific paths and domains
1 parent 746723d commit 9dc2517

File tree

2 files changed

+18
-4
lines changed

2 files changed

+18
-4
lines changed

.changeset/green-cameras-jam.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@labdigital/federated-token": patch
3+
---
4+
5+
Fix removing cookies for specific paths and domains

src/tokensource/cookies.ts

Lines changed: 13 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -54,13 +54,22 @@ export class CookieTokenSource implements TokenSource {
5454
}
5555

5656
deleteAccessToken(response: Response<any, Record<string, any>>): void {
57-
response.clearCookie(this.cookieNames.accessToken);
58-
response.clearCookie(this.cookieNames.accessTokenHash);
57+
response.clearCookie(this.cookieNames.accessToken, {
58+
domain: this.options?.publicDomainFn?.(response.req),
59+
});
60+
response.clearCookie(this.cookieNames.accessTokenHash, {
61+
domain: this.options?.privateDomainFn?.(response.req),
62+
});
5963
}
6064

6165
deleteRefreshToken(response: Response<any, Record<string, any>>): void {
62-
response.clearCookie(this.cookieNames.refreshToken);
63-
response.clearCookie(this.cookieNames.refreshTokenExist);
66+
response.clearCookie(this.cookieNames.refreshToken, {
67+
path: this.options.refreshTokenPath,
68+
domain: this.options?.privateDomainFn?.(response.req),
69+
});
70+
response.clearCookie(this.cookieNames.refreshTokenExist, {
71+
domain: this.options?.publicDomainFn?.(response.req),
72+
});
6473
}
6574

6675
getAccessToken(request: Request): string {

0 commit comments

Comments
 (0)