removing tls environment var (#1151)

* removing var

* fixing unused variable - CI failure

Author: marytlf

pkg/resources/management.cattle.io/v3/project/validator.go

@@ -148,7 +148,7 @@ func (a *admitter) admitUpdate(oldProject, newProject *v3.Project) (*admissionv1
 
 }
 
-func (a *admitter) admitCommonCreateUpdate(oldProject, newProject *v3.Project) (*admissionv1.AdmissionResponse, error) {
+func (a *admitter) admitCommonCreateUpdate(_, newProject *v3.Project) (*admissionv1.AdmissionResponse, error) {
 	projectQuota := newProject.Spec.ResourceQuota
 	nsQuota := newProject.Spec.NamespaceDefaultResourceQuota
 	containerLimit := newProject.Spec.ContainerDefaultResourceLimit

pkg/server/server.go

@@ -42,7 +42,6 @@ const (
 	webhookPortEnvKey       = "CATTLE_PORT"
 	webhookURLEnvKey        = "CATTLE_WEBHOOK_URL"
 	allowedCNsEnv           = "ALLOWED_CNS"
-	ignoreTLSHandshakeError = "IGNORE_TLS_HANDSHAKE_ERROR"
 )
 
 var caFile = filepath.Join(os.TempDir(), "k8s-webhook-server", "client-ca", "ca.crt")
@@ -151,7 +150,6 @@ func listenAndServe(ctx context.Context, clients *clients.Clients, validators []
 			return fmt.Errorf("failed to decode webhook port value '%s': %w", portStr, err)
 		}
 	}
-	ignoreTLSHandErrorVal, _ := strconv.ParseBool(os.Getenv(ignoreTLSHandshakeError))
 	return server.ListenAndServe(ctx, webhookHTTPSPort, webhookHTTPPort, router, &server.ListenOpts{
 		Secrets:       clients.Core.Secret(),
 		CertNamespace: namespace,
@@ -165,7 +163,7 @@ func listenAndServe(ctx context.Context, clients *clients.Clients, validators []
 			TLSConfig: tlsConfig,
 		},
 		DisplayServerLogs:       true,
-		IgnoreTLSHandshakeError: ignoreTLSHandErrorVal,
+		IgnoreTLSHandshakeError: true,
 	})
 }