-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Updated docs for the improved Google OIDC connector #9907
Conversation
Should I temporarily remove the video banner until a new video is made? |
|
||
Within GSuite to access the Manage API client access go to Security -> Settings. Navigate to Advanced Settings and open Manage API client access. Put the client ID in the Client Name field and the below permissions in the API scopes as a single comma separated line. Press Authorize. | ||
- Configure [domain-wide delegation](https://admin.google.com/ac/owl/domainwidedelegation) for your newly-created service account | ||
![domain-wide delegation](../../../img/googleoidc/domainwidedelegation.png) |
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.
This image looks a bit awkward in the middle of a sentence. We should either move it below this paragraph or start a new bullet point below the image.
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.
I had two remaining nits, but I've approved so my review doesn't block you.
Co-authored-by: Paul Gottschling <paul.gottschling@goteleport.com> Co-authored-by: Zac Bergquist <zmb3@users.noreply.github.com>
0274578
to
515b58c
Compare
515b58c
to
5953f12
Compare
* Updated docs for the improved Google OIDC connector * Apply suggestions from code review Co-authored-by: Paul Gottschling <paul.gottschling@goteleport.com> Co-authored-by: Zac Bergquist <zmb3@users.noreply.github.com> * Use "code" blocks for copyable commands * whitespace * Consistency with punctuation and avoid splitting paragraphs with images. * Updated error message matching the current code * Bump minimum version for OIDCConnector v3 to 8.1.2 * More fixes Co-authored-by: Paul Gottschling <paul.gottschling@goteleport.com> Co-authored-by: Zac Bergquist <zmb3@users.noreply.github.com>
Updated docs for