Skip to content

Commit

Permalink
test: use EC cert property now that it exists
Browse files Browse the repository at this point in the history
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 authored and targos committed Mar 27, 2019
1 parent ea42514 commit 9c83002
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions test/parallel/test-tls-multi-key.js
Expand Up @@ -160,9 +160,7 @@ function test(options) {
version: 'TLSv1/SSLv3'
});
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();
}));
Expand Down

0 comments on commit 9c83002

Please sign in to comment.