LDAP Plugin

phenomlab

@baris thanks but I'm not sure I fully understand this. Would you mind taking a look if you have time ?

Thanks

<baris>

I forked it here https://github.com/NodeBB-Community/nodebb-plugin-node-ldap, the commit with the fixes https://github.com/NodeBB-Community/nodebb-plugin-node-ldap/commit/764d8c316b19a349371044be76a8ba5a396d2597.

Let me know if it works then I can publish it to npm as @nodebb/nodebb-plugin-node-ldap

phenomlab

@baris getting a slew of errors as below

2022-10-31T15:43:29.836Z [4567/355627] - warn:    [plugins/nodebb-plugin-node-ldap] The plugin.json field "library" is deprecated. Please use the package.json field "main" instead.
2022-10-31T15:43:29.837Z [4567/355627] - warn: [plugins] Unable to load library for: nodebb-plugin-node-ldap
2022-10-31T15:43:29.837Z [4567/355627] - error: Error: Cannot find module './user'
Require stack:
- /home/etp/domains/kb.etp.local/nodebb/src/plugins/index.js
- /home/etp/domains/kb.etp.local/nodebb/src/meta/configs.js
- /home/etp/domains/kb.etp.local/nodebb/src/meta/index.js
- /home/etp/domains/kb.etp.local/nodebb/src/start.js
- /home/etp/domains/kb.etp.local/nodebb/require-main.js
- /home/etp/domains/kb.etp.local/nodebb/app.js
    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:985:15)
    at Function.Module._load (node:internal/modules/cjs/loader:833:27)
    at Module.require (node:internal/modules/cjs/loader:1057:19)
    at /home/etp/domains/kb.etp.local/nodebb/node_modules/nodebb-plugin-node-ldap/index.js:4:40
    at Object.<anonymous> (/home/etp/domains/kb.etp.local/nodebb/node_modules/nodebb-plugin-node-ldap/index.js:345:2)
    at Module._compile (node:internal/modules/cjs/loader:1155:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1209:10)
    at Module.load (node:internal/modules/cjs/loader:1033:32)
    at Function.Module._load (node:internal/modules/cjs/loader:868:12)
    at Module.require (node:internal/modules/cjs/loader:1057:19)

2022-10-31T15:43:29.847Z [4567/355627] - warn: [plugins/load] The following plugins may not be compatible with your version of NodeBB. This may cause unintended behaviour or crashing. In the event of an unresponsive NodeBB caused by this plugin, run `./nodebb reset -p PLUGINNAME` to disable it.
  * nodebb-rewards-essentials
  * nodebb-plugin-node-ldap

<baris>

Did you check index.js for the proper require statements? They should look like

 var user = require.main.require('./src/user'),
        groups = require.main.require('./src/groups'),
        meta = require.main.require('./src/meta'),
        db = require.main.require('./src/database'),
        passport = require.main.require('passport'),

No module.parent.require('./user'), seems like you are still using an old version.

phenomlab

Yes, I uninstalled the old one from NPM and the new one from GIT

phenomlab

@baris for clarity, I removed the older plugin, and used npm install git+https://github.com/NodeBB-Community/nodebb-plugin-node-ldap.git

phenomlab

I have a directory inside nodebb_mobules called nodebb-plugin-node-ldap-new and the below variables inside index.js

    var user = require.main.require('./src/user'),
        groups = require.main.require('./src/groups'),
        meta = require.main.require('./src/meta'),
        db = require.main.require('./src/database'),
        passport = require.main.require('passport'),
        async = require.main.require('async'),
        local_strategy = require.main.require('passport-local').Strategy,
        winston = require.main.require('winston'),
        ldapjs = require('ldapjs');
    const controllers = require.main.require('./src/controllers');

<baris>

nodebb-plugin-node-ldap-new this folder probably isn't used if the active plugin is nodebb-plugin-node-ldap. What does ./nodebb plugins show? You should place the fork on nodebb-community into nodebb-plugin-node-ldap and try again.

phenomlab

@baris thanks. I did actually do this the first time round but it's still the same.

<baris>

I published @nodebb/nodebb-plugin-node-ldap version is 2.0.1, deactivate and uninstall the other ones then install this one with npm i @nodebb/nodebb-plugin-node-ldap and let me know if you get any errors. This version if for nodebb 2.x.

phenomlab

@baris many thanks. I'll give this a try tomorrow am and let you know.

phenomlab

@baris the extension works in terms of no errors on install or activation, but doesn't work in the sense of actually logging in

Below are the errors generated when attempting to login via LDAP or AD

2022-11-01T09:49:46.122Z [4567/400206] - error: uncaughtException: input string cannot be empty
AssertionError [ERR_ASSERTION]: input string cannot be empty
    at Object.parse (/home/etp/domains/kb.etp.local/nodebb/node_modules/ldap-filter/lib/index.js:246:12)
    at Object.parseString (/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/node_modules/ldapjs/lib/filters/index.js:179:27)
    at Client.search (/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/node_modules/ldapjs/lib/client/client.js:569:30)
    at /home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/index.js:190:41
    at /home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/index.js:152:17
    at callbackWrapper (/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/node_modules/ldapjs/lib/client/client.js:292:5)
    at sendResult (/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/node_modules/ldapjs/lib/client/client.js:1195:12)
    at messageCallback (/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/node_modules/ldapjs/lib/client/client.js:1218:16)
    at Parser.onMessage (/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/node_modules/ldapjs/lib/client/client.js:884:14)
    at Parser.emit (node:events:513:28) {"date":"Tue Nov 01 2022 09:49:46 GMT+0000 (Coordinated Universal Time)","error":{"actual":false,"code":"ERR_ASSERTION","expected":true,"generatedMessage":false,"operator":"=="},"exception":true,"os":{"loadavg":[0.51,0.28,0.1],"uptime":423502.22},"process":{"argv":["/usr/bin/node","/home/etp/domains/kb.etp.local/nodebb/app.js"],"cwd":"/home/etp/domains/kb.etp.local/nodebb","execPath":"/usr/bin/node","gid":1001,"memoryUsage":{"arrayBuffers":19229585,"external":21369256,"heapTotal":87359488,"heapUsed":81645168,"rss":155594752},"pid":400206,"uid":1001,"version":"v16.18.0"},"stack":"AssertionError [ERR_ASSERTION]: input string cannot be empty\n    at Object.parse (/home/etp/domains/kb.etp.local/nodebb/node_modules/ldap-filter/lib/index.js:246:12)\n    at Object.parseString (/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/node_modules/ldapjs/lib/filters/index.js:179:27)\n    at Client.search (/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/node_modules/ldapjs/lib/client/client.js:569:30)\n    at /home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/index.js:190:41\n    at /home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/index.js:152:17\n    at callbackWrapper (/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/node_modules/ldapjs/lib/client/client.js:292:5)\n    at sendResult (/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/node_modules/ldapjs/lib/client/client.js:1195:12)\n    at messageCallback (/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/node_modules/ldapjs/lib/client/client.js:1218:16)\n    at Parser.onMessage (/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/node_modules/ldapjs/lib/client/client.js:884:14)\n    at Parser.emit (node:events:513:28)","trace":[{"column":12,"file":"/home/etp/domains/kb.etp.local/nodebb/node_modules/ldap-filter/lib/index.js","function":"Object.parse","line":246,"method":"parse","native":false},{"column":27,"file":"/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/node_modules/ldapjs/lib/filters/index.js","function":"Object.parseString","line":179,"method":"parseString","native":false},{"column":30,"file":"/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/node_modules/ldapjs/lib/client/client.js","function":"Client.search","line":569,"method":"search","native":false},{"column":41,"file":"/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/index.js","function":null,"line":190,"method":null,"native":false},{"column":17,"file":"/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/index.js","function":null,"line":152,"method":null,"native":false},{"column":5,"file":"/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/node_modules/ldapjs/lib/client/client.js","function":"callbackWrapper","line":292,"method":null,"native":false},{"column":12,"file":"/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/node_modules/ldapjs/lib/client/client.js","function":"sendResult","line":1195,"method":null,"native":false},{"column":16,"file":"/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/node_modules/ldapjs/lib/client/client.js","function":"messageCallback","line":1218,"method":null,"native":false},{"column":14,"file":"/home/etp/domains/kb.etp.local/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/node_modules/ldapjs/lib/client/client.js","function":"Parser.onMessage","line":884,"method":"onMessage","native":false},{"column":28,"file":"node:events","function":"Parser.emit","line":513,"method":"emit","native":false}]}

The below error is also shown when attempting to login

<baris>

I am not familiar with how the ldap plugin works, but the error seems to complain about some missing config. What values did you put in the ACP page? For reference the stack trace you posted points to this line in the plugin, so maybe master_config.base is missing or empty.

phenomlab

@baris figured out the missing config, thanks. Won't authenticate though, which I expect is down to the difference between LDAP and Windows AD

q16marvin

Hi, i tried your solution... i looks firstly "okay"... but when i try to login i got this:

screen:

error log:

2022-11-14T13:00:54.323Z [4567/83669] - error: uncaughtException: Cannot read properties of undefined (reading 'indexOf')
TypeError: Cannot read properties of undefined (reading 'indexOf')
    at Object.login (/root/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/index.js:257:23)
    at /root/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/index.js:212:57
    at callbackWrapper (/root/nodebb/node_modules/ldapjs/lib/client/client.js:292:5)
    at sendResult (/root/nodebb/node_modules/ldapjs/lib/client/client.js:1195:12)
    at messageCallback (/root/nodebb/node_modules/ldapjs/lib/client/client.js:1218:16)
    at Parser.onMessage (/root/nodebb/node_modules/ldapjs/lib/client/client.js:884:14)
    at Parser.emit (node:events:513:28)
    at Parser.write (/root/nodebb/node_modules/ldapjs/lib/messages/parser.js:107:8)
    at Socket.onData (/root/nodebb/node_modules/ldapjs/lib/client/client.js:871:22)
    at Socket.emit (node:events:513:28) {"date":"Mon Nov 14 2022 13:00:54 GMT+0000 (Koordinierte Weltzeit)","error":{},"exception":true,"os":{"loadavg":[0.34,0.13,0.04],"uptime":3028451.94},"process":{"argv":["/usr/bin/node","/root/nodebb/app.js"],"cwd":"/root/nodebb","execPath":"/usr/bin/node","gid":0,"memoryUsage":{"arrayBuffers":19463885,"external":21643628,"heapTotal":100155392,"heapUsed":92994872,"rss":172048384},"pid":83669,"uid":0,"version":"v18.12.1"},"stack":"TypeError: Cannot read properties of undefined (reading 'indexOf')\n    at Object.login (/root/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/index.js:257:23)\n    at /root/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/index.js:212:57\n    at callbackWrapper (/root/nodebb/node_modules/ldapjs/lib/client/client.js:292:5)\n    at sendResult (/root/nodebb/node_modules/ldapjs/lib/client/client.js:1195:12)\n    at messageCallback (/root/nodebb/node_modules/ldapjs/lib/client/client.js:1218:16)\n    at Parser.onMessage (/root/nodebb/node_modules/ldapjs/lib/client/client.js:884:14)\n    at Parser.emit (node:events:513:28)\n    at Parser.write (/root/nodebb/node_modules/ldapjs/lib/messages/parser.js:107:8)\n    at Socket.onData (/root/nodebb/node_modules/ldapjs/lib/client/client.js:871:22)\n    at Socket.emit (node:events:513:28)","trace":[{"column":23,"file":"/root/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/index.js","function":"Object.login","line":257,"method":"login","native":false},{"column":57,"file":"/root/nodebb/node_modules/@nodebb/nodebb-plugin-node-ldap/index.js","function":null,"line":212,"method":null,"native":false},{"column":5,"file":"/root/nodebb/node_modules/ldapjs/lib/client/client.js","function":"callbackWrapper","line":292,"method":null,"native":false},{"column":12,"file":"/root/nodebb/node_modules/ldapjs/lib/client/client.js","function":"sendResult","line":1195,"method":null,"native":false},{"column":16,"file":"/root/nodebb/node_modules/ldapjs/lib/client/client.js","function":"messageCallback","line":1218,"method":null,"native":false},{"column":14,"file":"/root/nodebb/node_modules/ldapjs/lib/client/client.js","function":"Parser.onMessage","line":884,"method":"onMessage","native":false},{"column":28,"file":"node:events","function":"Parser.emit","line":513,"method":"emit","native":false},{"column":8,"file":"/root/nodebb/node_modules/ldapjs/lib/messages/parser.js","function":"Parser.write","line":107,"method":"write","native":false},{"column":22,"file":"/root/nodebb/node_modules/ldapjs/lib/client/client.js","function":"Socket.onData","line":871,"method":"onData","native":false},{"column":28,"file":"node:events","function":"Socket.emit","line":513,"method":"emit","native":false}]}

<baris>

Can you try with 2.0.2, I fixed that undefined crash. I am not really familiar with how this plugin works so make sure you fill out the necessary info in the ACP page.
For example the plugin seems to look for the email in profile.mail but if there is an email_field set in the acp config then it will try to get the email from there.

https://github.com/NodeBB-Community/nodebb-plugin-node-ldap/blob/master/index.js#L253-L257

phenomlab

I found this listed in the NPM registry
https://github.com/Chrischenny/nodebb-plugin-node-ldap-ad

Can't get it to work though - crashes NodeBB every time.

EDIT - working now - I had the one amended by @baris still installed, so removed and all good from the stability perspective. Doesn't seem to work with AD though - at least, not for me - investigating.