Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Still active? #18

Open
uclajames opened this issue Oct 1, 2013 · 9 comments
Open

Still active? #18

uclajames opened this issue Oct 1, 2013 · 9 comments

Comments

@uclajames
Copy link

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
Copy link

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
Copy link
Author

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
Copy link

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
Copy link

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
Copy link

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
Copy link

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

@smithellis
Copy link

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
Copy link

tcnwd commented Sep 4, 2014

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

@smithellis
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants