Bind user LDAP models during the Login event

#581

Author: stevebauman

src/AdldapAuthServiceProvider.php

@@ -12,6 +12,7 @@
 use Illuminate\Support\Facades\Config;
 use Illuminate\Support\ServiceProvider;
 use Illuminate\Contracts\Hashing\Hasher;
+use Illuminate\Auth\Events\Login;
 use Illuminate\Auth\Events\Authenticated;
 
 class AdldapAuthServiceProvider extends ServiceProvider
@@ -63,6 +64,7 @@ public function register()
         // model to their Eloquent model upon authentication (if configured).
         // This allows us to utilize their LDAP model right
         // after authentication has passed.
+        Event::listen(Login::class, Listeners\BindsLdapUserModel::class);
         Event::listen(Authenticated::class, Listeners\BindsLdapUserModel::class);
 
         if ($this->isLogging()) {

src/Listeners/BindsLdapUserModel.php

@@ -6,21 +6,27 @@
 use Adldap\Laravel\Facades\Resolver;
 use Adldap\Laravel\Traits\HasLdapUser;
 use Illuminate\Support\Facades\Auth;
-use Illuminate\Auth\Events\Authenticated;
 use Illuminate\Contracts\Auth\Authenticatable;
 
 class BindsLdapUserModel
 {
     /**
      * Binds the LDAP user record to their model.
      *
-     * @param Authenticated $event
+     * @param mixed $event
      *
      * @return void
      */
-    public function handle(Authenticated $event)
+    public function handle($event)
     {
-        if ($this->isUsingAdldapProvider() && $this->canBind($event->user)) {
+        // Before we bind the users LDAP model, we will verify they are using the
+        // Adldap authentication provider, the required trait, and the
+        // users LDAP property has not already been set.
+        if (
+            $this->isUsingAdldapProvider()
+            && $this->canBind($event->user)
+            && is_null($event->user->ldap)
+        ) {
             $event->user->setLdapUser(
                 Resolver::byModel($event->user)
             );