fix: logout for all users to delete cookies
This commit is contained in:
parent
727f5c276e
commit
0dda336de1
@ -85,7 +85,6 @@ public class AuthController(IOptionsSnapshot<Admin> user, AuthService auth, Pass
|
|||||||
public async Task<ActionResult> Logout()
|
public async Task<ActionResult> Logout()
|
||||||
{
|
{
|
||||||
await auth.LogoutAsync(GetCookieParams(), HttpContext);
|
await auth.LogoutAsync(GetCookieParams(), HttpContext);
|
||||||
|
|
||||||
return Ok();
|
return Ok();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -193,6 +193,10 @@ public class AuthService(ICacheService cache, IAccessToken accessTokenService, I
|
|||||||
public async Task LogoutAsync(CookieOptionsParameters cookieOptions, HttpContext context, CancellationToken cancellation = default)
|
public async Task LogoutAsync(CookieOptionsParameters cookieOptions, HttpContext context, CancellationToken cancellation = default)
|
||||||
{
|
{
|
||||||
var requestContext = new RequestContextInfo(context, cookieOptions);
|
var requestContext = new RequestContextInfo(context, cookieOptions);
|
||||||
|
|
||||||
|
cookieOptions.DropCookie(context, CookieNames.AccessToken);
|
||||||
|
cookieOptions.DropCookie(context, CookieNames.RefreshToken);
|
||||||
|
|
||||||
var authTokenStruct = await cache.GetAsync<AuthToken>(GetAuthCacheKey(requestContext.Fingerprint), cancellation);
|
var authTokenStruct = await cache.GetAsync<AuthToken>(GetAuthCacheKey(requestContext.Fingerprint), cancellation);
|
||||||
|
|
||||||
if (authTokenStruct == null)
|
if (authTokenStruct == null)
|
||||||
@ -200,7 +204,5 @@ public class AuthService(ICacheService cache, IAccessToken accessTokenService, I
|
|||||||
|
|
||||||
await RevokeAccessToken(authTokenStruct.AccessToken);
|
await RevokeAccessToken(authTokenStruct.AccessToken);
|
||||||
await cache.RemoveAsync(requestContext.Fingerprint, cancellation);
|
await cache.RemoveAsync(requestContext.Fingerprint, cancellation);
|
||||||
cookieOptions.DropCookie(context, CookieNames.AccessToken);
|
|
||||||
cookieOptions.DropCookie(context, CookieNames.RefreshToken);
|
|
||||||
}
|
}
|
||||||
}
|
}
|
Loading…
x
Reference in New Issue
Block a user