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

Merge pull request #27 from ViliusS/stop-propagation

Browse Source 
Stop event propagation on empty username
This commit is contained in:
Andy Miller
2023-03-08 13:49:37 -07:00
committed by GitHub
parent 3998a5063e 2e60a75118
commit a921973929
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
login-ldap.php
View File

@@ -70,10 +70,11 @@ class LoginLDAPPlugin extends Plugin
public function userLoginAuthenticate(UserLoginEvent $event) public function userLoginAuthenticate(UserLoginEvent $event)
{ {
$credentials = $event->getCredentials(); $credentials = $event->getCredentials();
// empty username -> ignore // Fail early on empty username
if($credentials['username'] == ''){ if($credentials['username'] == ''){
$event->setStatus($event::AUTHENTICATION_FAILURE); $event->setStatus($event::AUTHENTICATION_FAILURE);
$event->stopPropagation();
return; return;
} }