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

Introduce user group management APIs #22

Merged
merged 20 commits into from
May 9, 2024
Merged

Conversation

codemonkeycxy
Copy link
Contributor

@codemonkeycxy codemonkeycxy commented Apr 16, 2024

What was changed

Introducing a new identity concept "UserGroup" with CRUD APIs that manage user group -> account/namespace access mapping

Why?

This is to enable the management of user group -> account/namespace access mapping by Temporal Cloud customers

  • Customers manage the user -> group mapping on their end
  • Customer sets up group -> account/namespace access mapping by calling these APIs
  • When users log in, Temporal Cloud uses the groups in user's token to perform authz

Checklist

  1. Closes

  2. How was this tested:

  1. Any docs updates needed?

@CLAassistant
Copy link

CLAassistant commented Apr 16, 2024

CLA assistant check
All committers have signed the CLA.

temporal/api/cloud/identity/v1/message.proto Outdated Show resolved Hide resolved
temporal/api/cloud/cloudservice/v1/service.proto Outdated Show resolved Hide resolved
temporal/api/cloud/cloudservice/v1/service.proto Outdated Show resolved Hide resolved
temporal/api/cloud/cloudservice/v1/request_response.proto Outdated Show resolved Hide resolved
temporal/api/cloud/cloudservice/v1/service.proto Outdated Show resolved Hide resolved
@codemonkeycxy codemonkeycxy changed the title Introduce group management APIs Introduce user group management APIs Apr 16, 2024
@codemonkeycxy codemonkeycxy merged commit 6e8267e into main May 9, 2024
3 checks passed
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.

5 participants