bitwarden/browser
1
0
Fork 0
mirror of https://github.com/bitwarden/browser synced 2026-02-10 05:30:01 +00:00
Code Issues Releases Wiki Activity

Make remember cache consistent with token cache and clear in all scenarios

Browse Source
This commit is contained in:
Alec Rippberger
2025-03-04 18:42:46 -06:00
parent 79f58dcee9
commit 6ab56027f2
2 changed files with 10 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
libs/auth/src/angular/two-factor-auth/two-factor-auth.component.ts
View File

@@ -359,11 +359,6 @@ export class TwoFactorAuthComponent implements OnInit, OnDestroy {
const authResult: AuthResult = await this.formPromise;
this.logService.info("Successfully submitted two factor token");
// Clear persisted data on successful login
if (this.twoFactorFormCacheService) {
await this.twoFactorFormCacheService.clearFormData();
}
await this.handleAuthResult(authResult);
} catch {
this.logService.error("Error submitting two factor token");
@@ -379,8 +374,8 @@ export class TwoFactorAuthComponent implements OnInit, OnDestroy {
// Persist current form data before navigating to another method
if (this.twoFactorFormCacheService) {
await this.twoFactorFormCacheService.saveFormData({
token: this.tokenFormControl.value || undefined,
remember: this.rememberFormControl.value ?? undefined,
token: undefined,
remember: undefined,
selectedProviderType: this.selectedProviderType,
emailSent: this.selectedProviderType === TwoFactorProviderType.Email,
});
@@ -401,10 +396,9 @@ export class TwoFactorAuthComponent implements OnInit, OnDestroy {
// Update the persisted provider type when a new one is chosen
if (this.twoFactorFormCacheService) {
const persistedData = await this.twoFactorFormCacheService.getFormData();
await this.twoFactorFormCacheService.saveFormData({
token: persistedData?.token || undefined,
remember: persistedData?.remember ?? undefined,
token: undefined,
remember: undefined,
selectedProviderType: response.type,
emailSent: false, // Reset email sent state when switching providers
});
@@ -487,6 +481,11 @@ export class TwoFactorAuthComponent implements OnInit, OnDestroy {
}
private async handleAuthResult(authResult: AuthResult) {
// Clear form cache
if (this.twoFactorFormCacheService) {
await this.twoFactorFormCacheService.clearFormData();
}
if (await this.handleMigrateEncryptionKey(authResult)) {
return; // stop login process
}