[PR #2623] [PM-2297] Login with trusted device (Flow 2) #3688

Closed
opened 2025-11-26 23:36:08 -06:00 by GiteaMirror · 0 comments
Owner

Original Pull Request: https://github.com/bitwarden/android/pull/2623

State: closed
Merged: Yes


Type of change

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

Objective

After an SSO Login with an organisation that has TDE enabled:

  • if the device is trusted, navigate to the users vault and decrypt the user's key with the device key.
  • if the device is not trusted, navigate to the device approval options screen.

Code changes

Additionally to the changes required to do the logic for the objective of the PR, navigations for the device approval options were also added to the iOS extensions.
Models were also updated with additional properties to match server responses.

Screenshots

https://github.com/bitwarden/mobile/assets/4648522/71ed138d-0736-4286-82c8-d955d04c3252

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
**Original Pull Request:** https://github.com/bitwarden/android/pull/2623 **State:** closed **Merged:** Yes --- ## Type of change - [ ] Bug fix - [X] 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--> After an SSO Login with an organisation that has TDE enabled: - if the device is trusted, navigate to the users vault and decrypt the user's key with the device key. - if the device is not trusted, navigate to the device approval options screen. ## 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--> Additionally to the changes required to do the logic for the objective of the PR, navigations for the device approval options were also added to the iOS extensions. Models were also updated with additional properties to match server responses. ## Screenshots <!--Required for any UI changes. Delete if not applicable--> https://github.com/bitwarden/mobile/assets/4648522/71ed138d-0736-4286-82c8-d955d04c3252 ## 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
GiteaMirror added the pull-request label 2025-11-26 23:36:08 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/android#3688