Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Merge pull request #170 from AlexVulaj/case-insensitive-csr-check
ignore case when checking csr hostnames
  • Loading branch information
openshift-merge-robot committed Sep 1, 2022
2 parents 3ee1fe4 + c3a6a76 commit 18dba4a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/controller/csr_check.go
Expand Up @@ -373,9 +373,9 @@ func authorizeServingCertWithMachine(machines []machinehandlerpkg.Machine, req *
var attemptedAddresses []string
var foundSan bool
for _, addr := range targetMachine.Status.Addresses {
switch corev1.NodeAddressType(addr.Type) {
switch addr.Type {
case corev1.NodeInternalDNS, corev1.NodeExternalDNS, corev1.NodeHostName:
if san == addr.Address {
if strings.EqualFold(san, addr.Address) {
foundSan = true
break
} else {
Expand Down

0 comments on commit 18dba4a

Please sign in to comment.