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
nodejs#24358

PR-URL: nodejs#26598
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Daijiro Wachi <[email protected]>
  • Loading branch information
sam-github authored and targos committed Mar 27, 2019
1 parent 9c1b09a commit 9277235
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions test/parallel/test-tls-multi-key.js
Original file line number Diff line number Diff line change
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 9277235

Please sign in to comment.