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

Feat: Implement Discovery support #1800

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

johanib
Copy link
Contributor

@johanib johanib commented Feb 5, 2025

Prior to this change, there were cases where it wasn't clear which
IdP end users should use. In these scenario's the users needed an IdP
which was not recognisable for them.

This change adds support for discovery IdP entries.
Which are additional names / ways of finding an IdP in the WAYF.
These can be configured in Manage.

A discovery requires at least an english name, but can also include
keywords or a custom logo, which is used on the consent page as well.

Resolves #1338

@johanib johanib requested a review from pablothedude February 5, 2025 14:37
@johanib johanib force-pushed the feature/1338-wayf-subtenants branch 2 times, most recently from 4d79d51 to 49e1679 Compare February 18, 2025 15:46
@johanib johanib changed the title WIP WAYF subtenants Feat: Implement Discovery support Feb 18, 2025
@johanib johanib marked this pull request as ready for review February 18, 2025 15:47
@johanib johanib force-pushed the feature/1338-wayf-subtenants branch 2 times, most recently from 151a528 to a505d71 Compare February 19, 2025 07:39
Prior to this change, there were cases where it wasn't clear which
IdP end users should use. In these scenario's the users needed an IdP
which was not recognisable for them.

This change adds support for discovery IdP entries.
Which are additional names / ways of finding an IdP in the WAYF.
These can be configured in Manage.

A discovery requires at least an english name, but can also include
keywords or a custom logo, which is used on the consent page as well.

Resolves #1338
@johanib johanib force-pushed the feature/1338-wayf-subtenants branch from a505d71 to 227fd15 Compare February 19, 2025 14:27
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.

Additional WAYF entries per IdP Endpoint with dedicated name, logo, keywords
1 participant