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

Use default verification instead of no verification #141

Merged
merged 1 commit into from
Aug 24, 2023

Conversation

jacobtomlinson
Copy link
Member

As noted in #139 (comment) it's better to set this to True to fall back to default verification instead of False for no verification.

@jacobtomlinson jacobtomlinson added kr8s auth Authenticating with Kubernetes labels Aug 24, 2023
@codecov
Copy link

codecov bot commented Aug 24, 2023

Codecov Report

Merging #141 (3c28f8d) into main (59d6a6b) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #141   +/-   ##
=======================================
  Coverage   95.59%   95.59%           
=======================================
  Files          24       24           
  Lines        2292     2292           
=======================================
  Hits         2191     2191           
  Misses        101      101           
Files Changed Coverage Δ
kr8s/_auth.py 91.89% <100.00%> (ø)

@jacobtomlinson jacobtomlinson merged commit fc9da85 into kr8s-org:main Aug 24, 2023
10 checks passed
@jacobtomlinson jacobtomlinson deleted the handle-missing-certs-2 branch August 24, 2023 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auth Authenticating with Kubernetes kr8s
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant