-
Notifications
You must be signed in to change notification settings - Fork 33
Add roles to SSO profile and directory user
#411
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
Conversation
Greptile OverviewGreptile SummaryAdded
Confidence Score: 5/5
Important Files ChangedFile Analysis
Sequence DiagramsequenceDiagram
participant API as WorkOS API
participant SDK as Ruby SDK
participant App as Application Code
Note over API,App: SSO Profile Flow
App->>SDK: SSO.profile(code)
SDK->>API: POST /sso/token
API-->>SDK: JSON response with role & roles
SDK->>SDK: Profile.initialize(json)
Note over SDK: Parses role (single) and<br/>roles (array) attributes
SDK-->>App: Profile object with role & roles
Note over API,App: Directory User Flow
App->>SDK: DirectorySync.get_user(id)
SDK->>API: GET /directory_users/:id
API-->>SDK: JSON response with role & roles
SDK->>SDK: DirectoryUser.initialize(json)
Note over SDK: Parses role (single) and<br/>roles (array) attributes
SDK->>SDK: to_json() merges base_attributes<br/>with authorization_attributes
SDK-->>App: DirectoryUser object with role & roles
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
7 files reviewed, 1 comment
Description
Adds the
rolesproperty to SSO profile and directory user to support multiple roles.Documentation
Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.
If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.