Skip to content

Commit

Permalink
Fix TLS-ALPN master breakage.
Browse files Browse the repository at this point in the history
golang#91 broke autocert tests because of the new OID this should bring the changes in line with master.
  • Loading branch information
jason-baker committed Feb 10, 2020
1 parent 6ca56c2 commit 77e8d1f
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions acme/autocert/autocert_test.go
Expand Up @@ -488,9 +488,8 @@ func startACMEServerStub(t *testing.T, tokenCert getCertificateFunc, domain stri
if err := crt.VerifyHostname(domain); err != nil {
t.Errorf("verifyTokenCert: %v", err)
}
// TODO: Update OID to the latest value 1.3.6.1.5.5.7.1.31
// See https://tools.ietf.org/html/draft-ietf-acme-tls-alpn-05#section-5.1
oid := asn1.ObjectIdentifier{1, 3, 6, 1, 5, 5, 7, 1, 30, 1}
oid := asn1.ObjectIdentifier{1, 3, 6, 1, 5, 5, 7, 1, 31}
for _, x := range crt.Extensions {
if x.Id.Equal(oid) {
// No need to check the extension value here.
Expand Down

0 comments on commit 77e8d1f

Please sign in to comment.