[PM-3096] Fix non discoverable passkey removal on moving to org#2637
Merged
fedemkr merged 1 commit intoPM-2378-handle-duplicate-passkey-on-moving-item-to-orgfrom Jul 21, 2023
Conversation
…crypting a cipher which was causing the passkey to be removed when moving to an org
mpbw2
approved these changes
Jul 21, 2023
fedemkr
merged commit a0112b3
into
PM-2378-handle-duplicate-passkey-on-moving-item-to-org
fedemkr
deleted the
vault/pm-3096/fix-non-discoverable-passkey-removal-on-moving-to-org
branch
fedemkr
added a commit
that referenced
this pull request
Jul 24, 2023
… to an organization (#2594) * PM-2378 Display error toast if there is a duplicate passkey when moving a cipher to an org * PM-3097 Fix issue when moving cipher with passkey to an org where the uniqueness should be taken into consideration on different passkeys types and also the Username (#2632) * PM-3096 Fix non-discoverable passkey to be taken into account when encrypting a cipher which was causing the passkey to be removed when moving to an org (#2637)
fedemkr
added a commit
that referenced
this pull request
Jul 26, 2023
* PM-1575 Added new models for Fido2Key * PM-1575 Added discoverable passkeys and WIP non-discoverable ones * PM-1575 Fix format * PM-1575 Added non-discoverable passkeys to login UI * PM-1575 Added copy application icon to Fido2Key UI * PM-1575 Updated bwi font with the updated passkey icon * PM-1575 For now just display Available for two-step login on non-discoverable passkey inside of a cipher login * PM-1575 Fix non-discoverable passkey visibility * PM-1575 remove Passkeys as a filter in the vault list * PM-1575 Display error toast if there is a duplicate passkey when moving a cipher to an org * Revert "PM-1575 Display error toast if there is a duplicate passkey when moving a cipher to an org" This reverts commit 78e6353. * [PM-2378] Display error toast on duplicate Passkey when moving cipher to an organization (#2594) * PM-2378 Display error toast if there is a duplicate passkey when moving a cipher to an org * PM-3097 Fix issue when moving cipher with passkey to an org where the uniqueness should be taken into consideration on different passkeys types and also the Username (#2632) * PM-3096 Fix non-discoverable passkey to be taken into account when encrypting a cipher which was causing the passkey to be removed when moving to an org (#2637)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Type of change
Objective
Fix when moving to org the non-discoverable passkey shouldn't be removed.
Code changes
EncryptablePropertiesto improve code reuse.Fido2Keyand also added the non-discoverable passkey (cipher.Login.Fido2Key) to be taken into account when encrypting a cipher.Before you submit
dotnet format --verify-no-changes) (required)