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

Login local user only and authentication by ldap server #232

Closed
wants to merge 2 commits into from

Conversation

patricklbs
Copy link

Add a code for three files for user local authentication. Because I need to centralize authentication by ldap and resctrictive authentication for local user only.

Active authentication ldap, thanks to agitate.
@ghost
Copy link

ghost commented Jan 19, 2013

Thank you for this, Patrick.

Nils - Any word on merging this code from Patrick? I really need the ability to restrict logins. Thanks!

@ghost
Copy link

ghost commented Jan 23, 2013

main.queries.php still needs this update in order for this to work:

-        } elseif (mysql_num_rows($row) == 0 && $ldap_connection == true) {
+        } elseif (mysql_num_rows($row) == 0 && $ldap_connection == true && isset($_SESSION['settings']['ldap_elusers']) && ($_SESSION['settings']['ldap_elusers'] == 0 )) {

@patricklbs
Copy link
Author

elseif (mysql_num_rows($row) == 0 && $ldap_connection == true&& isset($_SESSION['settings']['ldap_elusers']) && ($_SESSION['settings']['ldap_elusers'] == 0 )) {

Yes it's always on code twenty20 :)

@ghost
Copy link

ghost commented Jan 24, 2013

No, you didn't forget, it's in your commit. Somehow it didn't get merged into the 2.1.15 branch though.

patricklbs notifications@github.com wrote:

Yes it's right. I forget this ? So sorry I'm commit soon :)


Reply to this email directly or view it on GitHub:
#232 (comment)

nilsteampassnet added a commit that referenced this pull request Jan 27, 2013
 * list of bugs corrected: #242 - #254 - #244 - #247 - #256 - #250 - #254 - #248
   #243 - #252 - #232 - #240 - #260 - #259 - #262 - #251 - #236
 * MySQL hashing => todo
 * CSV importation
nilsteampassnet added a commit that referenced this pull request Jan 27, 2013
 * list of bugs corrected: #242 - #254 - #244 - #247 - #256 - #250 - #254 - #248
   #243 - #252 - #232 - #240 - #260 - #259 - #262 - #251 - #236
 * MySQL hashing => todo
 * CSV importation
nilsteampassnet added a commit that referenced this pull request Jan 27, 2013
 * list of bugs corrected: #242 - #254 - #244 - #247 - #256 - #250 - #254 - #248
   #243 - #252 - #232 - #240 - #260 - #259 - #262 - #251 - #236
 * MySQL hashing => todo
 * CSV importation
nilsteampassnet added a commit that referenced this pull request Jan 27, 2013
 * list of bugs corrected: #242 - #254 - #244 - #247 - #256 - #250 - #254 - #248
   #243 - #252 - #232 - #240 - #260 - #259 - #262 - #251 - #236
 * MySQL hashing => todo
 * CSV importation
nilsteampassnet added a commit that referenced this pull request Jan 27, 2013
 * list of bugs corrected: #242 - #254 - #244 - #247 - #256 - #250 - #254 - #248
   #243 - #252 - #232 - #240 - #260 - #259 - #262 - #251 - #236
 * MySQL hashing => todo
 * CSV importation
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

Successfully merging this pull request may close these issues.

2 participants