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
  • Loading branch information
sam-github committed Mar 11, 2019
1 parent 8f93c08 commit 5fa1d67
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 5fa1d67

Please sign in to comment.