Skip to content

LG-14325: Change from default to discouraged for authentication#11354

Merged
mdiarra3 merged 1 commit intomainfrom
LG-14325-move-to-discouraged-for-authentication
Oct 22, 2024
Merged

LG-14325: Change from default to discouraged for authentication#11354
mdiarra3 merged 1 commit intomainfrom
LG-14325-move-to-discouraged-for-authentication

Conversation

@mdiarra3
Copy link
Contributor

🎫 Ticket

Link to the relevant ticket:
LG-14325

🛠 Summary of changes

This makes it so when a user is leveraging webauthn as an MFA, it should not have to prompt user to enter in another credential before accepting the MFA.

📜 Testing Plan

When signing in with Security key and F/T Unlock, User should not be prompted to enter in PIN.

@aduth aduth requested a review from a team October 16, 2024 20:20
@aduth
Copy link
Contributor

aduth commented Oct 17, 2024

Nit: The title of the pull request doesn't seem accurate to its proposed changes, though the changelog itself looks correct.

Copy link
Contributor

@kevinsmaster5 kevinsmaster5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and tested locally 👍

@mdiarra3 mdiarra3 changed the title LG-14325: Change from default to preferred LG-14325: Change from default to discouraged for authentication Oct 17, 2024
@mdiarra3
Copy link
Contributor Author

whoops updated thanks for the callout @aduth

@mdiarra3 mdiarra3 merged commit d50b357 into main Oct 22, 2024
@mdiarra3 mdiarra3 deleted the LG-14325-move-to-discouraged-for-authentication branch October 22, 2024 15:08
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.

3 participants