Skip to content

LG-12578 active non biometric user can sign in to non biometric sp SAML#10276

Merged
theabrad merged 4 commits intomainfrom
abrad-lg-12578-saml-pending-profile
Mar 22, 2024
Merged

LG-12578 active non biometric user can sign in to non biometric sp SAML#10276
theabrad merged 4 commits intomainfrom
abrad-lg-12578-saml-pending-profile

Conversation

@theabrad
Copy link
Contributor

🎫 Ticket

LG-12578

🛠 Summary of changes

If a user has an active non biometric profile and a pending biometric profile, they will be redirected to the SP with the non biometric attributes if the SP does not request biometrics.

📜 Testing Plan

  • Go through identity-saml-sinatra proofing without a biometric requirement. Finish the entire flow.
  • Go through proofing with a biometric requirement. At the phone step choose to receive a letter
  • Attempt to go through proofing via an SP without a biometric requirement
  • Verify that you are redirected to the SP.

changelog: Internal, IdV, Active non biometric user sign in to non biometric sp SAML

If a user has an active non biometric profile and a pending biometric
profile, they will be redirected to the SP with the non biometric
attributes if the SP does not request biometrics.
@theabrad theabrad requested a review from a team March 21, 2024 15:31
Copy link
Contributor

@jmax-gsa jmax-gsa left a comment

Choose a reason for hiding this comment

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

LGTM

@theabrad theabrad merged commit 3ffdd80 into main Mar 22, 2024
@theabrad theabrad deleted the abrad-lg-12578-saml-pending-profile branch March 22, 2024 15:46
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