[PR #3192] [MERGED] [PM-7690] Unable to decrypt vault with MP after SSO authentication #23461

Closed
opened 2026-04-16 23:32:16 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/bitwarden/android/pull/3192
Author: @andrebispo5
Created: 4/25/2024
Status: Merged
Merged: 4/26/2024
Merged by: @andrebispo5

Base: mainHead: auth/pm-7690/sso-mp-unlock-bug


📝 Commits (1)

  • 0df46f3 [PM-7690] Fix login with master password vault unlock after SSO TDE decryption options

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 src/Core/Pages/Accounts/LoginSsoPage.xaml.cs (+1 -1)

📄 Description

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

Fix bug where users could not use master password as decryption option method for vault unlock.

Code changes

AppOptions were not set on navigation making the app throw an exception when trying to set _appOptions.HasJustLoggedInOrUnlocked = true; on vault unlock in LockPage

Screenshots

Before you submit

  • Please check for formatting errors (dotnet format --verify-no-changes) (required)
  • Please add unit tests where it makes sense to do so (encouraged but not required)
  • If this change requires a documentation update - notify the documentation team
  • If this change has particular deployment requirements - notify the DevOps team

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/bitwarden/android/pull/3192 **Author:** [@andrebispo5](https://github.com/andrebispo5) **Created:** 4/25/2024 **Status:** ✅ Merged **Merged:** 4/26/2024 **Merged by:** [@andrebispo5](https://github.com/andrebispo5) **Base:** `main` ← **Head:** `auth/pm-7690/sso-mp-unlock-bug` --- ### 📝 Commits (1) - [`0df46f3`](https://github.com/bitwarden/android/commit/0df46f3f3fb328f9acc027823cec58f5c5a0c0df) [PM-7690] Fix login with master password vault unlock after SSO TDE decryption options ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `src/Core/Pages/Accounts/LoginSsoPage.xaml.cs` (+1 -1) </details> ### 📄 Description ## Type of change - [X] Bug fix - [ ] New feature development - [ ] Tech debt (refactoring, code cleanup, dependency upgrades, etc) - [ ] Build/deploy pipeline (DevOps) - [ ] Other ## Objective <!--Describe what the purpose of this PR is. For example: what bug you're fixing or what new feature you're adding--> Fix bug where users could not use master password as decryption option method for vault unlock. ## Code changes <!--Explain the changes you've made to each file or major component. This should help the reviewer understand your changes--> <!--Also refer to any related changes or PRs in other repositories--> AppOptions were not set on navigation making the app throw an exception when trying to set `_appOptions.HasJustLoggedInOrUnlocked = true;` on vault unlock in `LockPage` ## Screenshots <!--Required for any UI changes. Delete if not applicable--> ## Before you submit - Please check for formatting errors (`dotnet format --verify-no-changes`) (required) - Please add **unit tests** where it makes sense to do so (encouraged but not required) - If this change requires a **documentation update** - notify the documentation team - If this change has particular **deployment requirements** - notify the DevOps team --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-16 23:32:16 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/android#23461