verdnatura-mirror
/
node-ldapjs
mirror of https://github.com/ldapjs/node-ldapjs.git
0
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Update guide.md 

Dear authors, many thanks for your project. When following your guide, the code crashed when i tried to add a user from the ldif file. I needed the code to change as seen above (the object includes a "attrs" key).
This commit is contained in:
maltewirz 2020-12-10 14:36:17 +01:00 committed by GitHub
parent 3909066af6
commit 97ed08861a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
docs/guide.md
View File

@ -400,10 +400,10 @@ the following code in as another handler (you'll need a
`var spawn = require('child_process').spawn;` at the top of your file): `var spawn = require('child_process').spawn;` at the top of your file):
server.add('ou=users, o=myhost', pre, function(req, res, next) { server.add('ou=users, o=myhost', pre, function(req, res, next) {
if (!req.dn.rdns[0].cn) if (!req.dn.rdns[0].attrs.cn)
return next(new ldap.ConstraintViolationError('cn required')); return next(new ldap.ConstraintViolationError('cn required'));
if (req.users[req.dn.rdns[0].cn]) if (req.users[req.dn.rdns[0].attrs.cn])
return next(new ldap.EntryAlreadyExistsError(req.dn.toString())); return next(new ldap.EntryAlreadyExistsError(req.dn.toString()));
var entry = req.toObject().attributes; var entry = req.toObject().attributes;
@ -487,10 +487,10 @@ Let's confirm he got added with an ldapsearch:
As before, here's a breakdown of the code: As before, here's a breakdown of the code:
server.add('ou=users, o=myhost', pre, function(req, res, next) { server.add('ou=users, o=myhost', pre, function(req, res, next) {
if (!req.dn.rdns[0].cn) if (!req.dn.rdns[0].attrs.cn)
return next(new ldap.ConstraintViolationError('cn required')); return next(new ldap.ConstraintViolationError('cn required'));
if (req.users[req.dn.rdns[0].cn]) if (req.users[req.dn.rdns[0].attrs.cn])
return next(new ldap.EntryAlreadyExistsError(req.dn.toString())); return next(new ldap.EntryAlreadyExistsError(req.dn.toString()));
var entry = req.toObject().attributes; var entry = req.toObject().attributes;
@ -532,13 +532,13 @@ RFC, so appending, removing, or replacing a single attribute is pretty natural.
Go ahead and add the following code into your source file: Go ahead and add the following code into your source file:
server.modify('ou=users, o=myhost', pre, function(req, res, next) { server.modify('ou=users, o=myhost', pre, function(req, res, next) {
if (!req.dn.rdns[0].cn || !req.users[req.dn.rdns[0].cn]) if (!req.dn.rdns[0].attrs.cn || !req.users[req.dn.rdns[0].attrs.cn])
return next(new ldap.NoSuchObjectError(req.dn.toString())); return next(new ldap.NoSuchObjectError(req.dn.toString()));
if (!req.changes.length) if (!req.changes.length)
return next(new ldap.ProtocolError('changes required')); return next(new ldap.ProtocolError('changes required'));
var user = req.users[req.dn.rdns[0].cn].attributes; var user = req.users[req.dn.rdns[0].attrs.cn].attributes;
var mod; var mod;
for (var i = 0; i < req.changes.length; i++) { for (var i = 0; i < req.changes.length; i++) {
@ -594,7 +594,7 @@ Delete is pretty straightforward. The client gives you a dn to delete, and you
delete it :). Add the following code into your server: delete it :). Add the following code into your server:
server.del('ou=users, o=myhost', pre, function(req, res, next) { server.del('ou=users, o=myhost', pre, function(req, res, next) {
if (!req.dn.rdns[0].cn || !req.users[req.dn.rdns[0].cn]) if (!req.dn.rdns[0].attrs.cn || !req.users[req.dn.rdns[0].attrs.cn])
return next(new ldap.NoSuchObjectError(req.dn.toString())); return next(new ldap.NoSuchObjectError(req.dn.toString()));
var userdel = spawn('userdel', ['-f', req.dn.rdns[0].cn]); var userdel = spawn('userdel', ['-f', req.dn.rdns[0].cn]);