Skip to content

Add centralized login sample app using OIDC client #64

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.

Sign up for GitHub

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

Jump to bottom
Merged
ancheetah merged 4 commits into from
Dec 17, 2025
Merged

Add centralized login sample app using OIDC client #64

ancheetah merged 4 commits into from
Dec 17, 2025

Conversation

@ancheetah
Copy link
Contributor

@ancheetah ancheetah commented Aug 20, 2025 •
edited
Loading

@ancheetah ancheetah added the do not merge Do not merge label Aug 20, 2025
@ancheetah ancheetah force-pushed the SDKS-4055-oidc-sample branch 2 times, most recently from 2290031 to de4a1fa Compare August 20, 2025 18:09
@ancheetah ancheetah requested review from cerebrl and ryanbas21 August 20, 2025 18:12
ryanbas21
ryanbas21 previously approved these changes Aug 20, 2025
View reviewed changes
Copy link
Contributor

@ryanbas21 ryanbas21 left a comment

Choose a reason for hiding this comment

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

👍 thanks!

cerebrl
cerebrl previously approved these changes Aug 20, 2025
View reviewed changes
Copy link
Contributor

@cerebrl cerebrl left a comment

Choose a reason for hiding this comment

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

I think this is good. Now that I'm seeing this, I'm wondering if this should be a new directory or just a replacement for the old OIDC sample app. Since this is a draft and doesn't need to be merged now, we can discuss this when we are all back.

@ancheetah ancheetah dismissed stale reviews from cerebrl and ryanbas21 via e6092fd August 20, 2025 23:56
@ancheetah ancheetah marked this pull request as ready for review December 11, 2025 17:21
@ancheetah ancheetah removed the do not merge Do not merge label Dec 17, 2025
@ancheetah ancheetah force-pushed the SDKS-4055-oidc-sample branch from 2b9716b to 5f88e01 Compare December 17, 2025 01:19
@ancheetah ancheetah force-pushed the SDKS-4055-oidc-sample branch from 5f88e01 to 97d9158 Compare December 17, 2025 13:14
@ancheetah ancheetah merged commit 4d703ff into main Dec 17, 2025
16 of 32 checks passed
@ancheetah ancheetah deleted the SDKS-4055-oidc-sample branch December 17, 2025 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cerebrl cerebrl cerebrl approved these changes

@ryanbas21 ryanbas21 ryanbas21 left review comments

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ancheetah @cerebrl @ryanbas21