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

Add groups enrollment logic to be applied after users registration #793

Merged
merged 11 commits into from
Jul 25, 2024

Conversation

garaimanoj
Copy link
Contributor

@garaimanoj garaimanoj commented Jun 19, 2024

Properties to be added in application.yml

iam:
  registration:
    default-groups:
      - name: /atlas
        enrollment: INSERT
      - name: /privileged
        enrollment: GROUP_REQUEST

When setting up new user accounts, administrators often need to determine which group these accounts should belong to by default. Here’s what you need to know:

name (Default Group):

  • The term “name” represents the specific group that serves as the default destination for new accounts.

  • When a user account is created, it automatically becomes a member of this default group unless specified otherwise.

enrollment Types:

There are two enrollment types that determine how new accounts join the default group:

  • INSERT Enrollment: With this type, the system swiftly adds the new account to the default group during account creation.
    No additional steps or user interaction are required—the account slips seamlessly into the group.

  • GROUP_REQUEST Enrollment (Not implemented): Here, the process is a bit more interactive. New request to join the group will be sent to the admin.

Current Functionality:
As of now, the system has implemented the INSERT enrollment type.
When creating a new account, it automatically becomes part of the default group (“name”) if that group exists.

@garaimanoj garaimanoj changed the base branch from master to develop June 19, 2024 10:50
@garaimanoj garaimanoj self-assigned this Jun 19, 2024
@garaimanoj garaimanoj linked an issue Jun 24, 2024 that may be closed by this pull request
@garaimanoj garaimanoj marked this pull request as ready for review July 1, 2024 08:13
Copy link
Member

@enricovianello enricovianello left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The main changement is related to addind a DefaultGroup properties class, such as all the others already defined into IamProperties

Copy link
Member

@enricovianello enricovianello left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a minor cosmetic fix requested. Another thing we could add to this PR description the example of the application.yml configuration required (e.g. the part we removed).
This will make easier to understand it when reading new features from release notes.

@enricovianello enricovianello changed the title Issue 758 add default group to new account Add groups enrollment logic to be applied after users registration Jul 25, 2024
@enricovianello enricovianello merged commit 4eff6bc into develop Jul 25, 2024
5 checks passed
@enricovianello enricovianello deleted the issue-758-add-default-group-to-new-account branch July 25, 2024 13:13
Copy link

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

Successfully merging this pull request may close these issues.

New accounts should be created with a set of default groups
2 participants