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

Update identity provider list #2486

Closed
wants to merge 1 commit into from
Closed

Update identity provider list #2486

wants to merge 1 commit into from

Conversation

mogul
Copy link
Contributor

@mogul mogul commented Apr 3, 2024

NSF is no longer registered as an IdP, but many other agencies are.

Changes proposed in this pull request:

  • Remove NSF
  • Add many others

Security Considerations

[Note the any security considerations here, or make note of why there are none]

None, this is just a documentation update; the list of IdPs is already on the login page.

NSF is no longer registered, but many others are.
@mogul
Copy link
Contributor Author

mogul commented Apr 3, 2024

Note that the directions here should also remove NSF, and account (hah!) for the other IdPs in the list somehow. I don't know what restrictions there are/aren't, so I didn't try to do that here.

@markdboyd
Copy link
Contributor

@mogul I'm not sure if we should explicitly list all the integrated providers. The list will always be changing and it seems like it will lead to perpetual documentation rot

@mogul
Copy link
Contributor Author

mogul commented Jun 4, 2024

Up to you, I was just trying to unrot it.

@markdboyd
Copy link
Contributor

@mogul I'm going to opt to remove the explicit agency references instead. See #2512. Thanks for bringing this to our attention!

@markdboyd markdboyd closed this Jun 4, 2024
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