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

Add OIDC support #126

Merged
merged 2 commits into from
Jul 25, 2023
Merged

Add OIDC support #126

merged 2 commits into from
Jul 25, 2023

Conversation

jacobtomlinson
Copy link
Member

@jacobtomlinson jacobtomlinson commented Jul 25, 2023

Add minimal OIDC support. This will allow a token to be read from the auth-provider config section. However, it does not support refreshing the token if it has expired.

I'll handle optional token refreshing in a followup PR.

xref #125

@codecov
Copy link

codecov bot commented Jul 25, 2023

Codecov Report

Merging #126 (0ccf461) into main (47eeb24) will decrease coverage by 0.13%.
Report is 2 commits behind head on main.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##             main     #126      +/-   ##
==========================================
- Coverage   96.20%   96.07%   -0.13%     
==========================================
  Files          24       24              
  Lines        2212     2216       +4     
==========================================
+ Hits         2128     2129       +1     
- Misses         84       87       +3     
Files Changed Coverage Δ
kr8s/_auth.py 89.79% <25.00%> (-2.76%) ⬇️

@jacobtomlinson jacobtomlinson merged commit 7e6765c into kr8s-org:main Jul 25, 2023
11 of 12 checks passed
@jacobtomlinson jacobtomlinson deleted the oidc branch July 25, 2023 21:09
@jacobtomlinson jacobtomlinson mentioned this pull request Sep 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant