jannik/grav-plugin-login-ldap
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix issue #28 - Grav username defaults to the LDAP users full DN

Browse Source 
Additional fix assignment operator bug in error handling
This commit is contained in:
J.P. Pasnak
2025-11-22 21:30:07 -06:00
committed by GitHub
parent a3f0a2cf88
commit a3650e1477
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
login-ldap.php
View File

@@ -131,7 +131,7 @@ class LoginLDAPPlugin extends Plugin
$ldap->bind($username, $credentials['password']); $ldap->bind($username, $credentials['password']);
// Create Grav User // Create Grav User
$grav_user = User::load(strtolower($username)); $grav_user = User::load(strtolower($credentials['username']));
// Set defaults with only thing we know... username provided // Set defaults with only thing we know... username provided
$grav_user['login'] = $credentials['username']; $grav_user['login'] = $credentials['username'];
@@ -258,7 +258,7 @@ class LoginLDAPPlugin extends Plugin
$this->grav['log']->error('plugin.login-ldap: ['. $e->getCode() . '] ' . $username . ' - ' . $message); $this->grav['log']->error('plugin.login-ldap: ['. $e->getCode() . '] ' . $username . ' - ' . $message);
// Just return so other authenticators can take a shot... // Just return so other authenticators can take a shot...
if ($message = "Invalid credentials") { if ($message == "Invalid credentials") {
return; return;
} }