From 3c0007a21a0f5ba3c0bf267568d310f172af9f87 Mon Sep 17 00:00:00 2001 From: Dinis Vieira Date: Fri, 22 Mar 2024 15:29:12 +0000 Subject: [PATCH] [PM-7009] Improved exception messages for the Broadcast Service message callback function (#3091) * Improved exception messages for the Broadcast Service message callback function * Update src/Core/App.xaml.cs Co-authored-by: Federico Maccaroni * Update src/Core/App.xaml.cs Co-authored-by: Federico Maccaroni * Update src/Core/App.xaml.cs Additional Null Check Co-authored-by: Federico Maccaroni * Update src/Core/App.xaml.cs Additional Null Check Co-authored-by: Federico Maccaroni * Update src/Core/App.xaml.cs Additional Null Check Co-authored-by: Federico Maccaroni * Update src/Core/App.xaml.cs Additional Null Check Co-authored-by: Federico Maccaroni * Update src/Core/App.xaml.cs Additional Null Check Co-authored-by: Federico Maccaroni --------- Co-authored-by: Federico Maccaroni --- src/Core/App.xaml.cs | 257 +++++++++++++++++++++++-------------------- 1 file changed, 139 insertions(+), 118 deletions(-) diff --git a/src/Core/App.xaml.cs b/src/Core/App.xaml.cs index 2c63c252d..31a3ba8ca 100644 --- a/src/Core/App.xaml.cs +++ b/src/Core/App.xaml.cs @@ -9,6 +9,7 @@ using Bit.Core; using Bit.Core.Abstractions; using Bit.Core.Enums; using Bit.Core.Models.Data; +using Bit.Core.Models.Domain; using Bit.Core.Models.Response; using Bit.Core.Pages; using Bit.Core.Services; @@ -167,132 +168,153 @@ namespace Bit.App _accountsManager.Init(() => Options, this); - Bootstrap(); - _broadcasterService.Subscribe(nameof(App), async (message) => - { - try - { - if (message.Command == "showDialog") - { - var details = message.Data as DialogDetails; - var confirmed = true; - var confirmText = string.IsNullOrWhiteSpace(details.ConfirmText) ? - AppResources.Ok : details.ConfirmText; - await MainThread.InvokeOnMainThreadAsync(async () => - { - if (!string.IsNullOrWhiteSpace(details.CancelText)) - { - confirmed = await MainPage.DisplayAlert(details.Title, details.Text, confirmText, - details.CancelText); - } - else - { - await MainPage.DisplayAlert(details.Title, details.Text, confirmText); - } - _messagingService.Send("showDialogResolve", new Tuple(details.DialogId, confirmed)); - }); - } -#if IOS - else if (message.Command == AppHelpers.RESUMED_MESSAGE_COMMAND) - { - ResumedAsync().FireAndForget(); - } - else if (message.Command == "slept") - { - await SleptAsync(); - } -#endif - else if (message.Command == "migrated") - { - await Task.Delay(1000); - await _accountsManager.NavigateOnAccountChangeAsync(); - } - else if (message.Command == POP_ALL_AND_GO_TO_TAB_GENERATOR_MESSAGE || - message.Command == POP_ALL_AND_GO_TO_TAB_MYVAULT_MESSAGE || - message.Command == POP_ALL_AND_GO_TO_TAB_SEND_MESSAGE || - message.Command == POP_ALL_AND_GO_TO_AUTOFILL_CIPHERS_MESSAGE || - message.Command == DeepLinkContext.NEW_OTP_MESSAGE) - { - if (message.Command == DeepLinkContext.NEW_OTP_MESSAGE) - { - Options.OtpData = new OtpData((string)message.Data); - } + _broadcasterService.Subscribe(nameof(App), BroadcastServiceMessageCallbackAsync); - await MainThread.InvokeOnMainThreadAsync(async () => + Bootstrap(); + } + + private async void BroadcastServiceMessageCallbackAsync(Message message) + { + try + { + ArgumentNullException.ThrowIfNull(message); + if (message.Command == "showDialog") + { + var details = message.Data as DialogDetails; + ArgumentNullException.ThrowIfNull(details); + ArgumentNullException.ThrowIfNull(MainPage); + + var confirmed = true; + var confirmText = string.IsNullOrWhiteSpace(details.ConfirmText) ? + AppResources.Ok : details.ConfirmText; + await MainThread.InvokeOnMainThreadAsync(ShowDialogAction); + async Task ShowDialogAction() + { + if (!string.IsNullOrWhiteSpace(details.CancelText)) { - if (MainPage is TabsPage tabsPage) + confirmed = await MainPage.DisplayAlert(details.Title, details.Text, confirmText, + details.CancelText); + } + else + { + await MainPage.DisplayAlert(details.Title, details.Text, confirmText); + } + _messagingService.Send("showDialogResolve", new Tuple(details.DialogId, confirmed)); + } + } +#if IOS + else if (message.Command == AppHelpers.RESUMED_MESSAGE_COMMAND) + { + ResumedAsync().FireAndForget(); + } + else if (message.Command == "slept") + { + await SleptAsync(); + } +#endif + else if (message.Command == "migrated") + { + await Task.Delay(1000); + await _accountsManager.NavigateOnAccountChangeAsync(); + } + else if (message.Command == POP_ALL_AND_GO_TO_TAB_GENERATOR_MESSAGE || + message.Command == POP_ALL_AND_GO_TO_TAB_MYVAULT_MESSAGE || + message.Command == POP_ALL_AND_GO_TO_TAB_SEND_MESSAGE || + message.Command == POP_ALL_AND_GO_TO_AUTOFILL_CIPHERS_MESSAGE || + message.Command == DeepLinkContext.NEW_OTP_MESSAGE) + { + if (message.Command == DeepLinkContext.NEW_OTP_MESSAGE) + { + Options.OtpData = new OtpData((string)message.Data); + } + + await MainThread.InvokeOnMainThreadAsync(ExecuteNavigationAction); + async Task ExecuteNavigationAction() + { + if (MainPage is TabsPage tabsPage) + { + ArgumentNullException.ThrowIfNull(tabsPage.Navigation); + ArgumentNullException.ThrowIfNull(tabsPage.Navigation.ModalStack); + while (tabsPage.Navigation.ModalStack.Count > 0) { - while (tabsPage.Navigation.ModalStack.Count > 0) - { - await tabsPage.Navigation.PopModalAsync(false); - } - if (message.Command == POP_ALL_AND_GO_TO_AUTOFILL_CIPHERS_MESSAGE) - { - MainPage = new NavigationPage(new CipherSelectionPage(Options)); - } - else if (message.Command == POP_ALL_AND_GO_TO_TAB_MYVAULT_MESSAGE) - { - Options.MyVaultTile = false; - tabsPage.ResetToVaultPage(); - } - else if (message.Command == POP_ALL_AND_GO_TO_TAB_GENERATOR_MESSAGE) - { - Options.GeneratorTile = false; - tabsPage.ResetToGeneratorPage(); - } - else if (message.Command == POP_ALL_AND_GO_TO_TAB_SEND_MESSAGE) - { - tabsPage.ResetToSendPage(); - } - else if (message.Command == DeepLinkContext.NEW_OTP_MESSAGE) - { - tabsPage.ResetToVaultPage(); - await tabsPage.Navigation.PushModalAsync(new NavigationPage(new CipherSelectionPage(Options))); - } + await tabsPage.Navigation.PopModalAsync(false); + } + if (message.Command == POP_ALL_AND_GO_TO_AUTOFILL_CIPHERS_MESSAGE) + { + MainPage = new NavigationPage(new CipherSelectionPage(Options)); + } + else if (message.Command == POP_ALL_AND_GO_TO_TAB_MYVAULT_MESSAGE) + { + Options.MyVaultTile = false; + tabsPage.ResetToVaultPage(); + } + else if (message.Command == POP_ALL_AND_GO_TO_TAB_GENERATOR_MESSAGE) + { + Options.GeneratorTile = false; + tabsPage.ResetToGeneratorPage(); + } + else if (message.Command == POP_ALL_AND_GO_TO_TAB_SEND_MESSAGE) + { + tabsPage.ResetToSendPage(); + } + else if (message.Command == DeepLinkContext.NEW_OTP_MESSAGE) + { + tabsPage.ResetToVaultPage(); + ArgumentNullException.ThrowIfNull(tabsPage.Navigation); + await tabsPage.Navigation.PushModalAsync(new NavigationPage(new CipherSelectionPage(Options))); } - }); - } - else if (message.Command == "convertAccountToKeyConnector") - { - await MainThread.InvokeOnMainThreadAsync(async () => - { - await MainPage.Navigation.PushModalAsync( - new NavigationPage(new RemoveMasterPasswordPage())); - }); - } - else if (message.Command == Constants.ForceUpdatePassword) - { - await MainThread.InvokeOnMainThreadAsync(async () => - { - await MainPage.Navigation.PushModalAsync( - new NavigationPage(new UpdateTempPasswordPage())); - }); - } - else if (message.Command == Constants.ForceSetPassword) - { - await MainThread.InvokeOnMainThreadAsync(() => MainPage.Navigation.PushModalAsync( - new NavigationPage(new SetPasswordPage(orgIdentifier: (string)message.Data)))); - } - else if (message.Command == "syncCompleted") - { - await _configService.GetAsync(true); - } - else if (message.Command == Constants.PasswordlessLoginRequestKey - || message.Command == "unlocked" - || message.Command == AccountsManagerMessageCommands.ACCOUNT_SWITCH_COMPLETED) - { - lock (_processingLoginRequestLock) - { - // lock doesn't allow for async execution - CheckPasswordlessLoginRequestsAsync().Wait(); } } } - catch (Exception ex) + else if (message.Command == "convertAccountToKeyConnector") { - LoggerHelper.LogEvenIfCantBeResolved(ex); + ArgumentNullException.ThrowIfNull(MainPage); + await MainThread.InvokeOnMainThreadAsync(NavigateToRemoveMasterPasswordPageAction); + async Task NavigateToRemoveMasterPasswordPageAction() + { + await MainPage.Navigation.PushModalAsync( + new NavigationPage(new RemoveMasterPasswordPage())); + } } - }); + else if (message.Command == Constants.ForceUpdatePassword) + { + ArgumentNullException.ThrowIfNull(MainPage); + await MainThread.InvokeOnMainThreadAsync(NavigateToUpdateTempPasswordPageAction); + async Task NavigateToUpdateTempPasswordPageAction() + { + await MainPage.Navigation.PushModalAsync( + new NavigationPage(new UpdateTempPasswordPage())); + } + } + else if (message.Command == Constants.ForceSetPassword) + { + ArgumentNullException.ThrowIfNull(MainPage); + await MainThread.InvokeOnMainThreadAsync(NavigateToSetPasswordPageAction); + void NavigateToSetPasswordPageAction() + { + MainPage.Navigation.PushModalAsync( + new NavigationPage(new SetPasswordPage(orgIdentifier: (string)message.Data))); + } + } + else if (message.Command == "syncCompleted") + { + await _configService.GetAsync(true); + } + else if (message.Command == Constants.PasswordlessLoginRequestKey + || message.Command == "unlocked" + || message.Command == AccountsManagerMessageCommands.ACCOUNT_SWITCH_COMPLETED) + { + lock (_processingLoginRequestLock) + { + // lock doesn't allow for async execution + CheckPasswordlessLoginRequestsAsync().Wait(); + } + } + } + catch (Exception ex) + { + LoggerHelper.LogEvenIfCantBeResolved(ex); + } } private async Task CheckPasswordlessLoginRequestsAsync() @@ -307,7 +329,6 @@ namespace Bit.App { return; } - var notification = await _stateService.GetPasswordlessLoginNotificationAsync(); if (notification == null) {