Upstream/bitwarden-browser
1
0
Fork 0
mirror of https://github.com/bitwarden/browser.git synced 2024-11-11 10:10:25 +01:00
Code Issues Projects Releases Wiki Activity

[PM-12007] Fix vault timeout action logout with account switching (#11008)

Browse Source 
* Protect Against Toast Error

* Use `concatMap` Instead of `switchMap`
This commit is contained in:
Justin Baur 2024-09-12 15:59:33 -04:00 committed by GitHub
parent a31ecb18a1
commit fe96aa85f2
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
apps/browser/src/popup/app.component.ts
View File

@ -247,7 +247,7 @@ export class AppComponent implements OnInit, OnDestroy {
// Displaying toasts isn't super useful on the popup due to the reloads we do.
// However, it is visible for a moment on the FF sidebar logout.
private async displayLogoutReason(logoutReason: LogoutReason) {
let toastOptions: ToastOptions;
let toastOptions: ToastOptions | null = null;
switch (logoutReason) {
case "invalidSecurityStamp":
case "sessionExpired": {
@ -260,6 +260,11 @@ export class AppComponent implements OnInit, OnDestroy {
}
}
if (toastOptions == null) {
// We don't have anything to show for this particular reason
return;
}
this.toastService.showToast(toastOptions);
}
}

4
libs/common/src/services/vault-timeout/vault-timeout.service.ts
View File

@ -1,4 +1,4 @@
import { combineLatest, filter, firstValueFrom, map, switchMap, timeout } from "rxjs";
import { combineLatest, concatMap, filter, firstValueFrom, map, timeout } from "rxjs";
import { LogoutReason } from "@bitwarden/auth/common";
import { LogService } from "@bitwarden/common/platform/abstractions/log.service";
@ -79,7 +79,7 @@ export class VaultTimeoutService implements VaultTimeoutServiceAbstraction {
this.accountService.activeAccount$,
this.accountService.accountActivity$,
]).pipe(
switchMap(async ([activeAccount, accountActivity]) => {
concatMap(async ([activeAccount, accountActivity]) => {
const activeUserId = activeAccount?.id;
for (const userIdString in accountActivity) {
const userId = userIdString as UserId;