From 675eff0886d3e1e79468ed7b6fa4c3d8617795ed Mon Sep 17 00:00:00 2001 From: Martin Schuppert Date: Mon, 15 Jul 2024 15:08:42 +0200 Subject: [PATCH] [rabbitmq] remove CommonName from tls cert The CommonName has a max length of 64 bytes. Since the CommonName value is already in the DnsNames of the certificate it is not required. --- pkg/openstack/rabbitmq.go | 3 --- 1 file changed, 3 deletions(-) diff --git a/pkg/openstack/rabbitmq.go b/pkg/openstack/rabbitmq.go index 1aade0a54..55a67e1f2 100644 --- a/pkg/openstack/rabbitmq.go +++ b/pkg/openstack/rabbitmq.go @@ -272,13 +272,10 @@ func reconcileRabbitMQ( } tlsCert := "" - commonName := fmt.Sprintf("%s.%s", hostname, ClusterInternalDomain) - if instance.Spec.TLS.PodLevel.Enabled { certRequest := certmanager.CertificateRequest{ IssuerName: instance.GetInternalIssuer(), CertName: fmt.Sprintf("%s-svc", rabbitmq.Name), - CommonName: &commonName, Hostnames: hostnames, Subject: &certmgrv1.X509Subject{ Organizations: []string{fmt.Sprintf("%s.%s", rabbitmq.Namespace, ClusterInternalDomain)},