search

designbyfront / LDAP-Authentication-for-ExpressionEngine

An ExpressionEngine Extension that allows the authentication of users via LDAP. LDAP details are copied to the EE database before standard MySQL authentication is performed. If user is not found on LDAP, MySQL authentication will still be performed (useful for EE users not in LDAP)
http://www.designbyfront.com
30 stars 16 forks source link

Still active? #18

Open uclajames opened 11 years ago

uclajames commented 11 years ago

Is this module still active? I'm looking at using it for a project that requires LDAP authentication. But if it is no longer compatible with the latest EE version (2.7 at the moment), that might not be the best plan.

lightfuze commented 11 years ago

Since Design by Front was acquired by Monotype/Typcast this project hasn't moved.

We've been using it with EE 2.5.5, but planning an upgrade to 2.7. Are you receiving specific error messages with 2.7?

uclajames commented 11 years ago

I haven't tried it yet with 2.7. It is for a project I'm scoping out. I'm just trying to figure out if it is a viable option. Are you using it with 2.5.5 without any issues?

lightfuze commented 11 years ago

I am currently using it in a production environment running EE 2.5.5 with no problems and just tested it today with a new install of EE 2.7.1. I had to make one change due to a deprecated function (pull request #19).

There' has not been any activity from the owner of this project in about three years, so I doubt my pull request will get merged... You can grab the code with the changes here: https://github.com/lightfuze/LDAP-Authentication-for-ExpressionEngine

allnyguy commented 10 years ago

Does this work with the newest version of EE 2.8.1 I have tried this code and still cannot get it to authenticate via LDAP... i get the error "Fatal error: Call to undefined function do_hash() in /var/www/html/system/expressionengine/third_party/nce_ldap/ext.nce_ldap.php on line 247" which is the do_hash that was added... when i use the "original" code, i get a different error pointing to line 246, which is what this code replaced?? any suggestions?

allnyguy commented 10 years ago

Found that it seems that do_hash might not work with my version of PHP and found that i should use just hash instead, so i updated the code to say hash instead of do_hash, but now i get the following error:

A PHP Error was encountered Severity: Warning Message: hash() expects at least 2 parameters, 1 given Filename: nce_ldap/ext.nce_ldap.php Line Number: 247

any suggestions?

allnyguy commented 10 years ago

got it... had to add sha1 to the beginning ( in line 247

smithellis commented 10 years ago

Works with 2.9. I can't figure out how to improve the user experience, though. First time users come and try to log in, and are sent to an error message page:

The following errors were encountered mbr_account_not_active

However, their account was created and is pending approval.

I'd love to have that be a friendly message telling them about the next steps instead.

Back to coding.....

tcnwd commented 10 years ago

Anyone using this on 2.9.0 ? We're getting a php error in cp/addons_extensions.php

smithellis commented 10 years ago

I have it working as I stated above, tcnwd. Can you speak to your error messages? What are you seeing/getting?