-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
support EC private key only for mTLS auth #5657
Conversation
@pktxu : Is this ready for review? |
note that I don't have access to an openshift environment, not sure what the failing checks mean |
I see thanks. The Windows / Java 11 build worked https://github.com/fabric8io/kubernetes-client/actions/runs/7264744508/job/19798701469 and the only change has been merging master in... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pktxu thank you for the pr. This looks very good, just a couple of questions about exception handling.
kubernetes-client-api/src/main/java/io/fabric8/kubernetes/client/internal/CertUtils.java
Outdated
Show resolved
Hide resolved
kubernetes-client-api/src/main/java/io/fabric8/kubernetes/client/internal/CertUtils.java
Outdated
Show resolved
Hide resolved
kubernetes-client-api/src/test/java/io/fabric8/kubernetes/client/internal/CertUtilsTest.java
Outdated
Show resolved
Hide resolved
kubernetes-client-api/src/test/java/io/fabric8/kubernetes/client/internal/CertUtilsTest.java
Outdated
Show resolved
Hide resolved
originally introduced in fabric8io#1314 later removed in aetion@9e1d434
Signed-off-by: Marc Nuri <[email protected]>
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
Description
Enable using pinniped provided EC private keys (without corresponding public key bits) to auth into a cluster.
Closes #5656
Type of change
test, version modification, documentation, etc.)
Checklist