Skip to content

Commit a473b8d

Browse files
committed
Move getPeerCertificate and getCipher to CryptoStream
1 parent 2ca63c8 commit a473b8d

3 files changed

Lines changed: 30 additions & 29 deletions

File tree

lib/tls.js

Lines changed: 24 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,26 @@ CryptoStream.prototype.end = function(err) {
6363
};
6464

6565

66+
CryptoStream.prototype.getPeerCertificate = function() {
67+
if (this.pair._ssl) {
68+
return this.pair._ssl.getPeerCertificate();
69+
} else {
70+
return null;
71+
}
72+
};
73+
74+
75+
CryptoStream.prototype.getCipher = function(err) {
76+
if (this.pair._ssl) {
77+
return this.pair._ssl.getCurrentCipher();
78+
} else {
79+
return null;
80+
}
81+
};
82+
83+
84+
85+
6686
/**
6787
* Provides a pair of streams to do encrypted communication.
6888
*/
@@ -357,24 +377,6 @@ SecurePair.prototype._error = function(err) {
357377
}
358378
};
359379

360-
361-
SecurePair.prototype.getPeerCertificate = function(err) {
362-
if (this._ssl) {
363-
return this._ssl.getPeerCertificate();
364-
} else {
365-
return null;
366-
}
367-
};
368-
369-
370-
SecurePair.prototype.getCipher = function(err) {
371-
if (this._ssl) {
372-
return this._ssl.getCurrentCipher();
373-
} else {
374-
return null;
375-
}
376-
};
377-
378380
// TODO: support anonymous (nocert) and PSK
379381

380382

@@ -606,11 +608,10 @@ exports.connect = function(port /* host, options, cb */) {
606608
socket.connect(port, host);
607609

608610
pair.on('secure', function() {
609-
console.log('client: connected+secure!');
610-
console.log('client pair.getPeerCertificate(): %j',
611-
pair.getPeerCertificate());
612-
console.log('client pair.getCipher(): %j',
613-
pair.getCipher());
611+
console.log('client cleartext.getPeerCertificate(): %j',
612+
cleartext.getPeerCertificate());
613+
console.log('client cleartext.getCipher(): %j',
614+
cleartext.getCipher());
614615

615616
if (cb) {
616617
cb(cleartext);

test/simple/test-securepair-client.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -89,10 +89,10 @@ function startClient() {
8989

9090
pair.on('secure', function() {
9191
console.log('client: connected+secure!');
92-
console.log('client pair.getPeerCertificate(): %j',
93-
pair.getPeerCertificate());
94-
console.log('client pair.getCipher(): %j',
95-
pair.getCipher());
92+
console.log('client pair.cleartext.getPeerCertificate(): %j',
93+
pair.cleartext.getPeerCertificate());
94+
console.log('client pair.cleartext.getCipher(): %j',
95+
pair.cleartext.getCipher());
9696
setTimeout(function() {
9797
pair.cleartext.write('hello\r\n');
9898
}, 500);

test/simple/test-securepair-server.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,8 @@ var server = net.createServer(function(socket) {
3535
pair.on('secure', function() {
3636
log('connected+secure!');
3737
pair.cleartext.write('hello\r\n');
38-
log(pair.getPeerCertificate());
39-
log(pair.getCipher());
38+
log(pair.cleartext.getPeerCertificate());
39+
log(pair.cleartext.getCipher());
4040
});
4141

4242
pair.cleartext.on('data', function(data) {

0 commit comments

Comments
 (0)