Skip to content

Skip select account screen on return visits#5637

Merged
orenyk merged 1 commit intomainfrom
oyk-transparent-step-up
Nov 24, 2021
Merged

Skip select account screen on return visits#5637
orenyk merged 1 commit intomainfrom
oyk-transparent-step-up

Conversation

@orenyk
Copy link
Contributor

@orenyk orenyk commented Nov 23, 2021

Why: this doesn't make sense for the step up flows since it's disruptive to users.

Currently, when a user authenticates with an active Login.gov session,
they are taken to an account selection screen as per the OIDC spec. In
order to provide a more seamless "step-up" flow from IAL1 to IAL2, we
are going to only take a user to that screen on their first visit to a
given SP, and skip it on subsequent visits.

@orenyk orenyk requested a review from jmhooper November 23, 2021 20:49
@orenyk orenyk force-pushed the oyk-transparent-step-up branch from afb08a5 to 72c8576 Compare November 23, 2021 22:16
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM

Currently, when a user authenticates with an active Login.gov session,
they are taken to an account selection screen as per the OIDC spec. In
order to provide a more seamless "step-up" flow from IAL1 to IAL2, we
are going to only take a user to that screen on their first visit to a
given SP, and skip it on subsequent visits.
@orenyk orenyk force-pushed the oyk-transparent-step-up branch from 72c8576 to f25005a Compare November 23, 2021 23:41
@orenyk orenyk merged commit 59cc5a4 into main Nov 24, 2021
@orenyk orenyk deleted the oyk-transparent-step-up branch November 24, 2021 00:11
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