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

ref: Tidy up how we process OIDC config #1740

Merged

Callisto13 merged 1 commit into main from refactor-auth-some-more

Mar 17, 2022

Contributor

Callisto13 commented Mar 16, 2022 •

edited

It was starting to look a little hairy in cmd/gitops-server/cmd/cmd.go so I have:

Removed a completely redundant type
Shuffled some config generation under the auth package
Streamlined a couple of signatures that were looking way too busy
Updated some auth tests to match the rest of the repo


          ref: Tidy up how we process OIDC config

b7916dc

It was starting to look a little hairy in cmd/gitops-server/cmd/cmd.go

Callisto13 added the exclude from release notes label

Callisto13 requested review from yitsushi and luizbafilho

March 16, 2022 17:02

yitsushi approved these changes

View reviewed changes

Callisto13 merged commit 24a07ce into main

Callisto13 deleted the refactor-auth-some-more branch

March 17, 2022 09:56

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment