event names for searchEntry && searchReference are 'broken'

[hillar]

node-ldapjs/lib/client/client.js

Line 1208 in b64ed9f

let event = msg.constructor.name

if constructor.name is not same as instanceof events will get 'wrong name'

[roubert]

This issue can cause real problems when using a module bundler such as rollup.js, which can cause these classes to be renamed to names that no longer correspond to the event names that should be used, thereby breaking any code that expects ldapjs to produce these event names as documented.

The linked pull request looks good to me, it solves the problem that I've encountered.

[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.