Skip to content

Commit 7f772de

Browse files
committed
chore: simplify masque certificate verification
1 parent 7bdeaab commit 7f772de

1 file changed

Lines changed: 3 additions & 14 deletions

File tree

transport/masque/masque.go

Lines changed: 3 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -66,25 +66,14 @@ func PrepareTlsConfig(privKey *ecdsa.PrivateKey, peerPubKey *ecdsa.PublicKey, sn
6666
// WARN: SNI is usually not for the endpoint, so we must skip verification
6767
InsecureSkipVerify: true,
6868
// we pin to the endpoint public key
69-
VerifyPeerCertificate: func(rawCerts [][]byte, verifiedChains [][]*x509.Certificate) error {
70-
if len(rawCerts) == 0 {
71-
return nil
72-
}
73-
69+
VerifyConnection: func(cs tls.ConnectionState) error {
7470
var err error
75-
for _, v := range rawCerts {
76-
cert, er := x509.ParseCertificate(v)
77-
if er != nil {
78-
err = errors.Join(err, er)
79-
continue
80-
}
81-
82-
if er = verfiyCert(cert); er != nil {
71+
for _, cert := range cs.PeerCertificates {
72+
if er := verfiyCert(cert); er != nil {
8373
err = errors.Join(err, er)
8474
continue
8575
}
8676
}
87-
8877
return err
8978
},
9079
}

0 commit comments

Comments
 (0)