Skip to content
This repository was archived by the owner on May 23, 2022. It is now read-only.

Token overview (SOFTWARE-5077) #941

Merged

Conversation

matyasselmeci
Copy link
Contributor

No description provided.

@matyasselmeci matyasselmeci requested a review from brianhlin March 14, 2022 05:15
@matyasselmeci
Copy link
Contributor Author

I still need to write some more background information on OAuth2 and OIDC... and also I haven't actually previewed this in mkdocs yet... but I think it's ready for a first review.

Copy link
Member

@brianhlin brianhlin left a comment

Choose a reason for hiding this comment

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

Looks very thorough! I think we've got what we need for now and can add the background stuff later. See additional comments.

@matyasselmeci matyasselmeci force-pushed the pr/SOFTWARE-5077.moartokens branch from aac46de to 469a94b Compare March 15, 2022 22:34
Copy link
Member

@brianhlin brianhlin left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@brianhlin brianhlin merged commit 9f3d8c6 into opensciencegrid:master Mar 15, 2022
@matyasselmeci matyasselmeci deleted the pr/SOFTWARE-5077.moartokens branch March 15, 2022 22:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants