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

Update cg-ui UAA client for dashboard project #839

Merged
merged 2 commits into from
May 2, 2024
Merged

Conversation

hursey013
Copy link
Contributor

@hursey013 hursey013 commented Apr 22, 2024

Changes proposed in this pull request:

  • Add cloud_controller.write and cloud_controller.read in addition to cloud_controller.admin for matching user scopes
  • Removes scim.me and profile as they are not needed

security considerations

None

bosh/opsfiles/clients.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@jameshochadel jameshochadel left a comment

Choose a reason for hiding this comment

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

I presume the profile and scim.me removal is intentional — if yes, looks good to me!

@jduss4
Copy link

jduss4 commented May 2, 2024

Yeah, as far as I can tell those aren't in the UAA / CF documentation and I suspect they were leftover from something paas-admin was doing! They don't appear to affect what we're doing, at any rate. Thank you!

@hursey013 hursey013 merged commit 675d30e into main May 2, 2024
2 checks passed
@hursey013 hursey013 deleted the bh-cgui-update branch May 2, 2024 15:58
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.

3 participants