Documentation shows false implementation

[OrbiterToad]

Hello There i just got started with ldapjs and there doesn't seem to be a fine example of how to use starttls
https://github.com/ldapjs/node-ldapjs/blob/master/docs/client.md#starttls

Doc says usage is starttls(options, controls, callback)

But example says

const opts = {
  ca: [fs.readFileSync('mycacert.pem')]
};

client.starttls(opts, (err, res) => {
  assert.ifError(err);

  // Client communication now TLS protected
});

But doesnt seem to be an optional param:

Client.prototype.starttls = function starttls (options, controls, callback, _bypass) {
  //...
}

how do i properly create tls upgrade with the starttls method? or better how do i create/use the control?

[jsumners]

The control is added by the starttls method:

node-ldapjs/lib/client/client.js

Lines 744 to 748 in 9143456

	const req = new ExtendedRequest({
	requestName: '1.3.6.1.4.1.1466.20037',
	requestValue: null,
	controls: controls
	})

[jsumners]

👋

On February 22, 2023, we released version 3 of this library. As a result, we are closing this issue/pull request.

Please see issue #839 for more information, including how to proceed if you feel this closure is in error.