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

Refactor sign-out link logic for AuthProvider's that support it #1456

Closed
1 task done
thekaveman opened this issue Jun 27, 2023 · 0 comments · Fixed by #1462
Closed
1 task done

Refactor sign-out link logic for AuthProvider's that support it #1456

thekaveman opened this issue Jun 27, 2023 · 0 comments · Fixed by #1462
Assignees

Comments

@thekaveman
Copy link
Member

thekaveman commented Jun 27, 2023

Only some AuthProvider support sign-out:

  • Login.gov for the senior flow does, because the IdG is talking directly to Login.gov
  • VA.gov for the veteran flow does not, because the IdG is talking to VA.gov and they are talking to 1 of 4 different IdPs, which is unknowable to Benefits

Acceptance Criteria

  • The Login.gov branded sign out button and sign out link only displays for the senior flow with Login.gov verification

Additional context

Slack thread and #1439 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

2 participants