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

[proposal] Reorganize IdP form #2471

Open
augustuswm opened this issue Sep 25, 2024 · 0 comments · May be fixed by #2511
Open

[proposal] Reorganize IdP form #2471

augustuswm opened this issue Sep 25, 2024 · 0 comments · May be fixed by #2511
Assignees
Labels
idea Just an idea, not something we should necessarily do
Milestone

Comments

@augustuswm
Copy link

Currently the IdP create form lists all of the form elements as a single column of form elements. We have seen multiple customers run in to confusion on exactly what each field is for, and I think we could partially improve this by reorganizing the form. Instead of a straight list of elements, we can group them based on object they are configuring (either the Service Provider (rack) or the IdP). As an initial idea I would propose:

General

  • Name
  • Description
  • Technical Contact Email

Service Provider

  • Service provider client ID
  • ACS URL
  • Single Logout (SLO) Url
  • Signed Requests (maybe a check-box that enables the cert / key fields)
    • Public Cert
    • Private Key

Identity Provider

  • Entity ID
  • Group attribute name
  • Metadata source
@augustuswm augustuswm added the idea Just an idea, not something we should necessarily do label Sep 25, 2024
@david-crespo david-crespo added this to the 12 milestone Oct 14, 2024
@charliepark charliepark linked a pull request Oct 21, 2024 that will close this issue
@charliepark charliepark self-assigned this Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
idea Just an idea, not something we should necessarily do
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants