Apply suggestions from code review
This commit is contained in:
parent
b860932b9b
commit
c2494eca00
|
@ -555,7 +555,7 @@ $ ldapsearch -H ldap://localhost:389 -x -D cn=demo,dc=example,dc=com \
|
|||
|
||||
This example demonstrates multi-threading via the `cluster` module utilizing a `net` server for initial socket receipt. An alternate example demonstrating use of the `connectionRouter` `serverOptions` hook is available in the `examples` directory.
|
||||
|
||||
```
|
||||
```js
|
||||
const cluster = require('cluster');
|
||||
const ldap = require('ldapjs');
|
||||
const net = require('net');
|
||||
|
|
|
@ -314,6 +314,8 @@ function Server (options) {
|
|||
}
|
||||
|
||||
self.newConnection = function (conn) {
|
||||
// TODO: make `newConnection` available on the `Server` prototype
|
||||
// https://github.com/ldapjs/node-ldapjs/pull/727/files#r636572294
|
||||
setupConnection(conn)
|
||||
log.trace('new connection from %s', conn.ldap.id)
|
||||
|
||||
|
|
Loading…
Reference in New Issue