LDAP Plugin

[phenomlab]

I spent a bit more time going through this, and changed some stuff around, but still can't get it to work. I have a requirement for this myself for a work related project.

@baris @julian any interest in making this plugin work ?

[baris]

I can take a look later today, the require statements should be require.main.require('./src/user') and this needs to be moved to the modules block

[phenomlab]

@baris Perfect. Thanks very much.

[phenomlab]

@baris did you get a chance to look into this ?

I made the following changes

\node_modules\nodebb-plugin-node-ldap\index.js

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

And in plugin.json

{
    "id": "nodebb-plugin-node-ldap",
    "name": "Node LDAP",
    "description": "An LDAP plugin",
	"url": "https://github.com/ink-blot/nodebb-plugin-node-ldap",
	"library": "./index.js",
	"hooks": [
		{ "hook": "static:app.load", "method": "init" },
		{ "hook": "action:auth.overrideLogin", "method": "override"	},
		{ "hook": "filter:config.get", "method": "get_config" },
		{ "hook": "filter:admin.header.build", "method": "admin" }
	],
    "modules": {
    	"acpScripts": "/lib/node_ldap.js"
    },
	"templates": "templates"
}

Still get the same errors though

Many thanks

[baris]

Haven't had a change yet, the modules block looks wrong though, it should be

"modules: {
  "../admin/plugins/nodebb_ldap.js": "lib/node_ldap.js"
}

Let me know if that works.

[phenomlab]

@baris Same error sadly

VM2161:2 Uncaught ReferenceError: require is not defined
    at <anonymous>:2:5
    at W (admin.min.js?v=mnm8h9os1sc:48:826)
    at Pt (admin.min.js?v=mnm8h9os1sc:48:49754)
    at r.fn.init.append (admin.min.js?v=mnm8h9os1sc:48:50965)
    at r.fn.init.<anonymous> (admin.min.js?v=mnm8h9os1sc:48:52055)
    at Te (admin.min.js?v=mnm8h9os1sc:48:33415)
    at r.fn.init.html (admin.min.js?v=mnm8h9os1sc:48:51735)
    at admin.min.js?v=mnm8h9os1sc:1:24108
(anonymous) @ VM2161:2
W @ admin.min.js?v=mnm8h9os1sc:48
Pt @ admin.min.js?v=mnm8h9os1sc:48
append @ admin.min.js?v=mnm8h9os1sc:48
(anonymous) @ admin.min.js?v=mnm8h9os1sc:48
Te @ admin.min.js?v=mnm8h9os1sc:48
html @ admin.min.js?v=mnm8h9os1sc:48
(anonymous) @ admin.min.js?v=mnm8h9os1sc:1
Promise.then (async)
(anonymous) @ admin.min.js?v=mnm8h9os1sc:1
(anonymous) @ admin.min.js?v=mnm8h9os1sc:1
Promise.then (async)
g @ admin.min.js?v=mnm8h9os1sc:1
(anonymous) @ admin.min.js?v=mnm8h9os1sc:1
success @ admin.min.js?v=mnm8h9os1sc:1
D @ admin.min.js?v=mnm8h9os1sc:48
fireWith @ admin.min.js?v=mnm8h9os1sc:48
tt @ admin.min.js?v=mnm8h9os1sc:51
(anonymous) @ admin.min.js?v=mnm8h9os1sc:51
load (async)
send @ admin.min.js?v=mnm8h9os1sc:51
ajax @ admin.min.js?v=mnm8h9os1sc:51
ajaxify.loadData @ admin.min.js?v=mnm8h9os1sc:1
ajaxify.go @ admin.min.js?v=mnm8h9os1sc:1
E @ admin.min.js?v=mnm8h9os1sc:1
(anonymous) @ admin.min.js?v=mnm8h9os1sc:1
dispatch @ admin.min.js?v=mnm8h9os1sc:48
me.handle @ admin.min.js?v=mnm8h9os1sc:48
admin.min.js?v=mnm8h9os1sc:1 error loading node_ldap
Error: Cannot find module './node_ldap'
    at https://kb.etp.local/assets/admin.min.js?v=mnm8h9os1sc:2:25033
    at async g (https://kb.etp.local/assets/admin.min.js?v=mnm8h9os1sc:1:33052)
    at async Promise.all (index 0)
    at async app.require (https://kb.etp.local/assets/admin.min.js?v=mnm8h9os1sc:2:54)
    at async https://kb.etp.local/assets/admin.min.js?v=mnm8h9os1sc:1:27039

[baris]

https://github.com/csturiale/nodebb-plugin-node-ldap/blob/master/templates/nodebb_ldap.tpl#L134-L142 this block also needs to be removed from the template. It's already in this file https://github.com/csturiale/nodebb-plugin-node-ldap/blob/master/lib/nodebb_ldap.js

[phenomlab]

@baris thanks. I'll have a look at this tomorrow.

Edit - no free time today sadly but will check as soon as I can

[phenomlab]

@baris I removed the duplicate block as you indicated above, but the "cannot find module" error still persists - the require error is now gone

[baris]

Admin template needs to be in templates/admin folder and update the res.render call to use admin/nodebb_ldap

[baris]

Actually should be admin/plugins folder check how its done in persona or quick start.

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