Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Account Deletion SSO with CME OTP parameter #1751

Merged
merged 1 commit into from
Feb 3, 2022

Conversation

fedemkr
Copy link
Member

@fedemkr fedemkr commented Feb 3, 2022

Type of change

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

Objective

Fix OTP parameter sent on account deletion of SSO with CME.

Code changes

  • DeleteAccountRequest.cs: Added OTP parameter
  • VerificationActionsFlowHelper: Added more logic to set the verification type on the parameters and also hash the password directly here
  • VerificationCodeViewModel: Set the verification type on the parameters
  • DeleteAccountViewModel.cs -> DeleteAccountActionFlowExecutioner: Updated it to consider passing the master password hash or the OTP parameter to the request.

Testing requirements

A user should be able to delete an account with master password and also with OTP (SSO with CME)

Before you submit

  • I have added unit tests where it makes sense to do so (encouraged but not required)
  • This change requires a documentation update (notify the documentation team)
  • This change has particular deployment requirements (notify the DevOps team)

@fedemkr fedemkr requested a review from a team February 3, 2022 20:07
@fedemkr fedemkr merged commit 31cdf40 into master Feb 3, 2022
@fedemkr fedemkr deleted the account-delete-cme-verification-fix branch February 3, 2022 20:46
fedemkr added a commit that referenced this pull request Feb 4, 2022
fedemkr added a commit that referenced this pull request Feb 7, 2022
* accountswitching:
  Autosync the updated translations (#1752)
  Fix delete account SSO with CME that the OTP parameter was being sent incorrectly to the server (#1751)
fedemkr added a commit that referenced this pull request Feb 11, 2022
* master: (34 commits)
  check email for null before authenticating (#1769)
  remove datepicker style workaround (#1768)
  Bump version to 2.16.2 (#1765)
  Improved code for periodic Autofocus on scan for better cancellation and task handlilng (#1764)
  Updated Sed expression for Android manifests (#1763)
  Bumped version to 2.16.1 (#1762)
  Add iOS Share Extension to our version bump automation (#1761)
  Bumped version to 2.16.0 (#1760)
  Improved Autofocus code on ScanPage for better cancellation and exception handling #1228 (#1759)
  [Help] Update links to new pattern (#1758)
  Client & Version headers (#1757)
  Autosync the updated translations (#1752)
  Fix delete account SSO with CME that the OTP parameter was being sent incorrectly to the server (#1751)
  [Icons] Fast follower changes (#1750)
  Removed punctuation on some string resources regarding send (#1747)
  Set background of biometricCell from theme (#1744)
  Cleared About information about push registration, just left it in DEBUG mode (#1742)
  Cleared Console.WriteLine for iOS push notifications issue (#1739)
  Autosync the updated translations (#1734)
  Update FA icons on LockPasswordViewController (#1729)
  ...
fedemkr added a commit that referenced this pull request Feb 14, 2022
* master:
  [Icons] - BUG - Update groupings icon for collections (#1773)
  Autosync the updated translations (#1766)
  check email for null before authenticating (#1769)
  remove datepicker style workaround (#1768)
  Bump version to 2.16.2 (#1765)
  Improved code for periodic Autofocus on scan for better cancellation and task handlilng (#1764)
  Updated Sed expression for Android manifests (#1763)
  Bumped version to 2.16.1 (#1762)
  Add iOS Share Extension to our version bump automation (#1761)
  Bumped version to 2.16.0 (#1760)
  Improved Autofocus code on ScanPage for better cancellation and exception handling #1228 (#1759)
  [Help] Update links to new pattern (#1758)
  Client & Version headers (#1757)
  Autosync the updated translations (#1752)
  Fix delete account SSO with CME that the OTP parameter was being sent incorrectly to the server (#1751)
  [Icons] Fast follower changes (#1750)
  Removed punctuation on some string resources regarding send (#1747)

# Conflicts:
#	src/Android/Properties/AndroidManifest.xml
#	src/iOS.Autofill/Info.plist
#	src/iOS.Extension/Info.plist
#	src/iOS.ShareExtension/Info.plist
#	src/iOS/Info.plist
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants