opendocman icon indicating copy to clipboard operation
opendocman copied to clipboard

Ldap ad/develop

Open HyTronix opened this issue 11 years ago • 1 comments

Hi Steve,

Please tell me I got this right this time... Git is confusing the heck out of me!

I think I followed your instructions to fork develop, then created my own feature branch, LDAP-AD/develop. I have pushed by changes to this branch, so now I submit a pull request like this, correct?

What I've got done here:

-Added a page to the installer for LDAP/AD options -Modified the config-sample.php template to accomodate this -Modified index.php to check for LDAP/AD users -Added ldap-users.inc with all functions in it -Fixed a bug I found in index.php (two arrays needed 0 index) and one in install/setup-config.php (parsing database table prefix) by adding it to my routines -Added a brief README.LDAP explaining things

Tested, works great here.

New functionality:

Will query LDAP/AD for a user account, and if the username/password match (by allowing a bind with them) a new user will be created in the ODM DB, if they don't already exist. If they do exist, all fields in the DB for that user get updated with values from LDAP/AD.

Let me know what you think, or if there's anything else I can do.

Thanks,

-John

HyTronix avatar Mar 30 '15 16:03 HyTronix

Please merge master into your branch to resolve the conflicts and try the pull request again (if you are done with it).

stephenlawrence avatar Oct 08 '15 01:10 stephenlawrence