Skip to content

Commit

Permalink
Switch to server name that includes full cluster domain to not break …
Browse files Browse the repository at this point in the history
…clients with partial SANs list (#1614)
  • Loading branch information
andrewstucki authored Dec 3, 2024
1 parent 926cfe0 commit 7736c4e
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions pkg/redpanda/client.go
Original file line number Diff line number Diff line change
Expand Up @@ -270,8 +270,7 @@ func certificatesFor(dot *helmette.Dot, cert string) (certSecret, certKey, clien

func tlsConfigFromDot(dot *helmette.Dot, listener redpanda.InternalTLS) (*tls.Config, error) {
namespace := dot.Release.Namespace
serviceName := redpanda.ServiceName(dot)
serverName := fmt.Sprintf("%s.%s.svc", serviceName, namespace)
serverName := redpanda.InternalDomain(dot)

rootCertName, rootCertKey, clientCertName := certificatesFor(dot, listener.Cert)

Expand Down

0 comments on commit 7736c4e

Please sign in to comment.