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

Adds RBAC support for Identity Center Accounts #49808

Merged
merged 8 commits into from
Dec 6, 2024
Merged

Conversation

tcsc
Copy link
Contributor

@tcsc tcsc commented Dec 5, 2024

  • Adds custom RoleMatchers for Identity Center resources
  • Disables label checking for Identity Center Account resources
  • Adds custom action checker that understands the generic
    KindIdentityCenter resource kind, and falls back from the
    requested resource kind to the generic one if not explicitly
    denied.

    For the purposes of the UI, Identity Center accounts and account
    assignments are treated like special Apps. This patch exposes
    Account Assignments to the UI via the Unified Resource Cache.

    Includes:
    - Generating an App resource from an Identity Center Account resource
    - General plumbing from backend through to cache and UI
@tcsc tcsc added no-changelog Indicates that a PR does not require a changelog entry backport/branch/v17 labels Dec 5, 2024
lib/services/role.go Outdated Show resolved Hide resolved
lib/services/identitycenter.go Outdated Show resolved Hide resolved
lib/auth/auth_with_roles.go Outdated Show resolved Hide resolved
lib/services/identitycenter.go Outdated Show resolved Hide resolved
Copy link
Contributor

@smallinsky smallinsky left a comment

Choose a reason for hiding this comment

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

Thanks for extracting the RBAC check to a separate PR. It looks much cleaner and it easy to review.

@tcsc tcsc force-pushed the tcsc/idc-account-rbac branch 2 times, most recently from 6b9b2b3 to 7fabd53 Compare December 5, 2024 12:05
 - Adds custom RoleMatchers for Identity Center resources
 - Disables label checking for Identity Center Account resources
 - Adds custom action checker that understands the generic
   `KindIdentityCenter` resource kind, and falls back from the
   requested resource kind to the generic one if not explicitly
   denied.
@tcsc tcsc force-pushed the tcsc/idc-account-rbac branch from 7fabd53 to 315148b Compare December 5, 2024 12:37
Base automatically changed from tcsc/idc-expose-accounts-as-apps to master December 5, 2024 12:43
@tcsc tcsc added this pull request to the merge queue Dec 6, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 6, 2024
@tcsc tcsc added this pull request to the merge queue Dec 6, 2024
Merged via the queue into master with commit de8e27f Dec 6, 2024
39 checks passed
@tcsc tcsc deleted the tcsc/idc-account-rbac branch December 6, 2024 01:19
@public-teleport-github-review-bot

@tcsc See the table below for backport results.

Branch Result
branch/v17 Failed

tcsc added a commit that referenced this pull request Dec 10, 2024
Backports #49808

 - Adds custom RoleMatchers for Identity Center resources
 - Disables label checking for Identity Center Account resources
 - Adds custom action checker that understands the generic
   `KindIdentityCenter` resource kind, and falls back from the
   requested resource kind to the generic one if not explicitly
   denied.
github-merge-queue bot pushed a commit that referenced this pull request Dec 10, 2024
* [v17] Adds RBAC support for Identity Center Accounts

Backports #49808

 - Adds custom RoleMatchers for Identity Center resources
 - Disables label checking for Identity Center Account resources
 - Adds custom action checker that understands the generic
   `KindIdentityCenter` resource kind, and falls back from the
   requested resource kind to the generic one if not explicitly
   denied.

* Update lib/services/role.go

Co-authored-by: Sakshyam Shah <[email protected]>

---------

Co-authored-by: Sakshyam Shah <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aws-iam-identity-center backport/branch/v17 no-changelog Indicates that a PR does not require a changelog entry size/md
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants