Skip to content
This repository was archived by the owner on May 14, 2024. It is now read-only.

Commit 3ca7e6d

Browse files
committed
Allow startTLS during client setup
1 parent 821569c commit 3ca7e6d

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

CHANGES.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@
2929
- Removed `spaced()` and `rndSpaced` from DN API
3030
- Fix parent/child rules regarding empty DNs
3131
- #254 Allow simple client bind with empty credentials
32+
- Add StartTLS support to client
3233

3334
## 0.7.1
3435

lib/client/client.js

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1119,14 +1119,17 @@ Client.prototype.connect = function connect() {
11191119
// socket before signalling successful connection.
11201120
// This setup needs to bypass the request queue since all other activity is
11211121
// blocked until the connection is considered fully established post-setup.
1122-
// Only allow bind/search for now.
1122+
// Only allow bind/search/starttls for now.
11231123
var basicClient = {
11241124
bind: function bindBypass(name, credentials, controls, callback) {
11251125
return self.bind(name, credentials, controls, callback, true);
11261126
},
11271127
search: function searchBypass(base, options, controls, callback) {
11281128
return self.search(base, options, controls, callback, true);
11291129
},
1130+
starttls: function starttlsBypass(options, controls, callback) {
1131+
return self.starttls(options, controls, callback, true);
1132+
},
11301133
unbind: self.unbind.bind(self)
11311134
};
11321135
vasync.forEachPipeline({

0 commit comments

Comments
 (0)