Upstream/bitwarden-browser
1
0
Fork 0
mirror of https://github.com/bitwarden/browser.git synced 2024-12-22 16:29:09 +01:00
Code Issues Projects Releases Wiki Activity

Defect/PM-2403 - Fix non-locally reproducible Master Password Invalid issue in Safari after SSO + 2FA login (#5531)

Browse Source 
* PM-2403 - (1) Fix issue with Safari's tab extension not closing after successful SSO Login + 2FA continue which sidesteps the inconsistent MP invalid issue (due to invalid KDF config settings on reload) -- Firefox + sidebar, opera + sidebar, chrome, safari, and edge tested (2) Refactor reload logic to exempt the current window from the reload logic as we really only need to reload sidebars so they end up on the lock screen vs staying on the login screen and we are just going to close the open extension running in the tab.  (3) Added comments

* PM-2403 - Per PR feedback, update reloadOpenWindows exemptCurrentHref filter to properly work

* PM-2403 - Per PR feedback, remove unneeded new method and replace with existing method closeBitwardenExtensionTab after testing on all browsers (apparently the chrome.tabs namespace is  supported in all major browsers that we support now)

* PM-2403 - Refactor sync to be kicked off earlier in the process b/c to make sure that closing the window does not abort it
This commit is contained in:
Jared Snider 2023-05-29 00:37:47 -04:00 committed by GitHub
parent f4cf53c308
commit d63a9a2d9f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 17 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
apps/browser/src/auth/popup/two-factor.component.ts
View File

@ -118,10 +118,20 @@ export class TwoFactorComponent extends BaseTwoFactorComponent {
this.route.queryParams.pipe(first()).subscribe(async (qParams) => { this.route.queryParams.pipe(first()).subscribe(async (qParams) => {
if (qParams.sso === "true") { if (qParams.sso === "true") {
super.onSuccessfulLogin = () => { super.onSuccessfulLogin = () => {
BrowserApi.reloadOpenWindows(); // This is not awaited so we don't pause the application while the sync is happening.
const thisWindow = window.open("", "_self"); // This call is executed by the service that lives in the background script so it will continue
thisWindow.close(); // the sync even if this tab closes.
return this.syncService.fullSync(true); const syncPromise = this.syncService.fullSync(true);
// Force sidebars (FF && Opera) to reload while exempting current window
// because we are just going to close the current window.
BrowserApi.reloadOpenWindows(true);
// We don't need this window anymore because the intent is for the user to be left
// on the web vault screen which tells them to continue in the browser extension (sidebar or popup)
BrowserApi.closeBitwardenExtensionTab();
return syncPromise;
}; };
} }
}); });

4
apps/browser/src/browser/browserApi.ts
View File

@ -227,10 +227,12 @@ export class BrowserApi {
} }
} }
static reloadOpenWindows() { static reloadOpenWindows(exemptCurrentHref = false) {
const currentHref = window.location.href;
const views = chrome.extension.getViews() as Window[]; const views = chrome.extension.getViews() as Window[];
views views
.filter((w) => w.location.href != null && !w.location.href.includes("background.html")) .filter((w) => w.location.href != null && !w.location.href.includes("background.html"))
.filter((w) => !exemptCurrentHref || w.location.href !== currentHref)
.forEach((w) => { .forEach((w) => {
w.location.reload(); w.location.reload();
}); });