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 OIDC GitHub docs and update the test #21774

Merged
merged 1 commit into from
Nov 30, 2021

Conversation

sberyozkin
Copy link
Member

Fixes #21743

This PR adds more content to the OIDC GitHub section and updates the test to verify the proposed doc changes.

CC @FroMage

@sberyozkin sberyozkin force-pushed the improve_oidc_github_docs branch from 991e39d to be8dfaf Compare November 29, 2021 13:49
Copy link
Contributor

@gastaldi gastaldi left a comment

Choose a reason for hiding this comment

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

Added some observations

@sberyozkin sberyozkin force-pushed the improve_oidc_github_docs branch from 0c23967 to 2dfd477 Compare November 29, 2021 16:56
@quarkus-bot

This comment has been minimized.

@sberyozkin sberyozkin force-pushed the improve_oidc_github_docs branch from 2dfd477 to 91b0008 Compare November 29, 2021 22:41
@sberyozkin
Copy link
Member Author

@gastaldi Did a minor clarification to the docs (suggested to use both JsonWebToken and UserInfo as a possibly simpler alternative - would make sense if more than a name is required)

@sberyozkin sberyozkin merged commit 31caccc into quarkusio:main Nov 30, 2021
@quarkus-bot quarkus-bot bot added this to the 2.6 - main milestone Nov 30, 2021
@sberyozkin sberyozkin deleted the improve_oidc_github_docs branch November 30, 2021 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve OIDC GitHub doc section
3 participants