-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
fix: remove deprecated default cipher for golang >=1.22 #17569
Conversation
Signed-off-by: Josh Wolf <[email protected]>
Have you run |
I see, golang/go#63413 |
@joshrwolf I'll mark this PR as a draft until you run |
FYI, I had go 1.22 installed and running the codegen locally gave me errors with packages downloaded with go 1.22. After installing 1.21, re-downloading all the local toolings in See kubernetes-sigs/controller-tools#888
|
Signed-off-by: Alexander Matyushentsev <[email protected]>
I took the liberty, run codegen and pushed changes to the branch ( |
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.
LGTM
* remove deprecated default cipher Signed-off-by: Josh Wolf <[email protected]> * regenerate CLI docs Signed-off-by: Alexander Matyushentsev <[email protected]> --------- Signed-off-by: Josh Wolf <[email protected]> Signed-off-by: Alexander Matyushentsev <[email protected]> Co-authored-by: Jann Fischer <[email protected]> Co-authored-by: Alexander Matyushentsev <[email protected]>
* remove deprecated default cipher Signed-off-by: Josh Wolf <[email protected]> * regenerate CLI docs Signed-off-by: Alexander Matyushentsev <[email protected]> --------- Signed-off-by: Josh Wolf <[email protected]> Signed-off-by: Alexander Matyushentsev <[email protected]> Co-authored-by: Jann Fischer <[email protected]> Co-authored-by: Alexander Matyushentsev <[email protected]>
Is this there in any released versions yet? @alexmt |
This is slated for 2.12; there's an RC out for it now. |
Closes #17571
newer versions of go remove the
TLS_RSA_WITH_AES_256_GCM_SHA384
as an available cipher.this PR simply removes it in favor of the existing default
TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384
, which remains a valid default option for the existing go toolchain version, and future onesChecklist: