-
Notifications
You must be signed in to change notification settings - Fork 0
add sso flow #119
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 sso flow #119
Conversation
# Conflicts: # openid-client/openid-client.go
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.
Pull Request Overview
This PR adds SSO (Single Sign-On) flow functionality to enable users to open a new web session in IAS using an SSO token. The implementation includes a new command handler that constructs a SAML SSO URL and opens it in the user's default browser based on their platform.
Key changes:
- Added a new
HandleSsoFlowfunction to process SSO tokens and launch browser-based authentication - Introduced
ssocommand and related flags (-redirect_uri,-sso) for SSO flow configuration - Added support for resource parameter override in token-exchange flows
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.
| File | Description |
|---|---|
| pkg/client/client.go | Implements the core SSO flow handler with platform-specific browser opening logic |
| openid-client/openid-client.go | Adds CLI command and flags for SSO functionality, integrates SSO flow into main execution path |
| README.md | Documents the new SSO command and related flags |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
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.
Pull Request Overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
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.
Pull Request Overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.