TypeError: Cannot convert undefined or null to object

[lfcastilhos]

Team, recompile my PCF code from 22/2/2023 and I got the error below.
Nothing was changed in my code, except redeploy and package update.
Also I see ldapjs was updated to version 3.0.0.
Anyone having same issue?

/home/vcap/deps/0/node_modules/activedirectory/lib/activedirectory.js:416
delete result.controls; // Remove the controls array returned as part of the SearchEntry
^
TypeError: Cannot convert undefined or null to object
at CorkedEmitter.onSearchEntry (/home/vcap/deps/0/node_modules/activedirectory/lib/activedirectory.js:416:5)
at CorkedEmitter.emit (node:events:513:28)
at CorkedEmitter.emit (/home/vcap/deps/0/node_modules/ldapjs/lib/corked_emitter.js:44:33)
at sendResult (/home/vcap/deps/0/node_modules/ldapjs/lib/client/client.js:1233:22)
at messageCallback (/home/vcap/deps/0/node_modules/ldapjs/lib/client/client.js:1253:14)
at Parser.onMessage (/home/vcap/deps/0/node_modules/ldapjs/lib/client/client.js:919:14)
at Parser.emit (node:events:513:28)
at Parser.write (/home/vcap/deps/0/node_modules/ldapjs/lib/messages/parser.js:135:8)
at TLSSocket.onData (/home/vcap/deps/0/node_modules/ldapjs/lib/client/client.js:875:22)
at TLSSocket.emit (node:events:513:28)
Node.js v18.12.11

[lfcastilhos]

No issues using ldapjs 2.3.3. looks like it's incompatible with ldapjs 3.0.0.;

[andreeib]

Indeed, adding an override in package.json fixes the issue for now:

  "overrides": {
    "activedirectory": {
      "ldapjs": "2.3.3"
    }
  }

[OzukaruCid]

Hi,, y added the override from adreeib, but still the `errror..!!

`{
"dependencies": {
"activedirectory": "^0.7.2"
}

,"overrides": {
  "activedirectory": {
     "ldapjs": "2.3.3"
   }

}
}`

[OzukaruCid]

it's work now,, i'm sorry still learning..

[willcallender]

I'm having the same issue, except that mine is persistent even after applying the fixes suggested here and in #60. The code that generates the error is

ad.findUser(username, (err, user) => {
  console.log('something');
  if (err) {
    console.log('ERROR: ' + JSON.stringify(err));
    return;
  }

  if (user) {
    console.log('User: ' + JSON.stringify(user));
  } else {
    console.log('User: ' + username + ' not found.');
  }
});

And my updated package.json

...
  "dependencies": {
    "activedirectory": "^0.7.2",
    "cors": "^2.8.5",
    "express": "^4.18.2",
    "moment": "^2.29.4",
    "pg": "^8.11.3"
  },
  "overrides": {
    "activedirectory": {
      "ldapjs": "2.3.3"
    }
  }

I get the same exact error as previous comments. The node server is running in portainer and the container has been destroyed and recreated. I have also rebooted the physical server with no effect. Commenting out the affected line in activedirectory.js bypasses the fatal error, but is unable to find the user. Any suggestions? Thanks!

[joaopenas]

Hello @willcallender I have been getting the same issue in the last 2 weeks and I have not solution for this.
I started change implementation to use ldapts library. If you want some help to change it I can help you ;)

[OzukaruCid]

Hola a todos: La implementación de ldap en mi proyecto es muy básico... tuve el mismo problema y estuve a punto de desistir pero instale activedirectory2... Y con eso quedó resuelto... Saludos El vie., 27 oct 2023 3:55 a. m., João Penas ***@***.***> escribió:

…

Hello @willcallender <https://github.com/willcallender> I have been getting the same issue in the last 2 weeks and I have not solution for this. I started change implementation to use ldapts library. If you want some help to change it I can help you ;) — Reply to this email directly, view it on GitHub <#246 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AKFFZCE26AKS7G6VKZ4QVQTYBNZJPAVCNFSM6AAAAAAVGBEDRKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBSGU2DSNJWGY> . You are receiving this because you commented.Message ID: ***@***.***>

[willcallender]

@OzukaruCid that worked immediately! Thank you!