Skip to content
Permalink
Browse files

test: use EC cert property now that it exists

Remove XXX, there has been an EC specific cert property since
#24358

PR-URL: #26598
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
  • Loading branch information...
sam-github committed Mar 11, 2019
1 parent 97a919b commit ed2c6965d2f901f3c786f9d24bcd57b2cd523611
Showing with 1 addition and 3 deletions.
  1. +1 −3 test/parallel/test-tls-multi-key.js
@@ -160,9 +160,7 @@ function test(options) {
version: 'TLSv1.2'
});
assert.strictEqual(ecdsa.getPeerCertificate().subject.CN, eccCN);
// XXX(sam) certs don't currently include EC key info, so depend on
// absence of RSA key info to indicate key is EC.
assert(!ecdsa.getPeerCertificate().exponent, 'not cert for an RSA key');
assert.strictEqual(ecdsa.getPeerCertificate().asn1Curve, 'prime256v1');
ecdsa.end();
connectWithRsa();
}));

0 comments on commit ed2c696

Please sign in to comment.
You can’t perform that action at this time.