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: initial creation of allOf group auth #555

Closed
wants to merge 5 commits into from

Conversation

UnicornChance
Copy link
Contributor

Description

Building out functionality for incorporating an allOf group authentication option type. This group auth is a list of groups defined by an application and for a user to authenticate to that app must have all of the groups in that list.

This new feature also makes sure that the anyOf type can be used with allOf for more complicated use cases.

Unit tests have been created and possibly a follow on PR for e2e tests. Docs also updated to describe the use case.

Related Issue

Relates to this uds-identity PR

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Other (security config, docs update, etc)

Checklist before merging

@UnicornChance UnicornChance linked an issue Jul 10, 2024 that may be closed by this pull request
@mjnagel
Copy link
Contributor

mjnagel commented Aug 8, 2024

Closing this for now - we aren't seeing any user requirements/requests for it. This could be reevaluated in the future.

@mjnagel mjnagel closed this Aug 8, 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.

allOf group auth
2 participants