-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Backport webhook.Server TLSOptions to v0.11 and v0.12? #2353
Labels
kind/support
Categorizes issue or PR as a support question.
Comments
/kind support |
k8s-ci-robot
added
the
kind/support
Categorizes issue or PR as a support question.
label
Sep 6, 2023
It's all good now, thank you so much for the help
…On Tue, Sep 5, 2023, 8:46 PM Troy Connor ***@***.***> wrote:
/kind support
@nathanperkins <https://github.com/nathanperkins> has this issue been
resolved with the PR linked above?
—
Reply to this email directly, view it on GitHub
<#2353 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACO3ZVYKEO2YWE5M4XEEGRLXY7WZXANCNFSM6AAAAAAYPR6APU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Thx for the confirmation /close |
@sbueringer: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
TLSOptions were added to webhook.Server in v0.13: 187187e
We need them to mitigate some security issues with the TLS configuration, but we don't want to bump controller-runtime minor versions in our older controller releases which are in maintenance for security fixes only.
Would you accept PRs to cherry pick these to v0.11 and v0.12 and cut patch releases?
The text was updated successfully, but these errors were encountered: