You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Similar to the flags for Webhook Port and Host, we should also expose flags for setting the minimum TLS version as well as the cipher suites to be used when creating the default webhook server.
Currently, to set the minimum TLS version, a hack would be to call manager.GetWebhookServer().TLSMinVersion before calling manager.Start(). To keep the behavior consistent, it makes sense to introduce new options in the manager.Options struct which allow doing the same.
The text was updated successfully, but these errors were encountered:
Currently, to set the minimum TLS version, a hack would be to call manager.GetWebhookServer().TLSMinVersion before calling manager.Start().
I don't think it's true and with this issue fixed we have many ways to get one thing done. AFAICS, without #2023, the webhook settings can be set like:
Similar to the flags for Webhook Port and Host, we should also expose flags for setting the minimum TLS version as well as the cipher suites to be used when creating the default webhook server.
Currently, to set the minimum TLS version, a hack would be to call
manager.GetWebhookServer().TLSMinVersion
before callingmanager.Start()
. To keep the behavior consistent, it makes sense to introduce new options in themanager.Options
struct which allow doing the same.The text was updated successfully, but these errors were encountered: