Catch bind exceptions on connection failures and log them

- Closes #295
- Binding failures will now be logged instead of being thrown for a
graceful fallback to eloquent

Author: stevebauman

src/AdldapServiceProvider.php

@@ -4,11 +4,13 @@
 
 use Adldap\Adldap;
 use Adldap\AdldapInterface;
+use Adldap\Auth\BindException;
 use Adldap\Connections\Provider;
 use Adldap\Schemas\SchemaInterface;
 use Adldap\Connections\ConnectionInterface;
 use Adldap\Laravel\Exceptions\ConfigurationMissingException;
 use Illuminate\Container\Container;
+use Illuminate\Support\Facades\Log;
 use Illuminate\Support\ServiceProvider;
 
 class AdldapServiceProvider extends ServiceProvider
@@ -93,9 +95,15 @@ protected function addProviders(Adldap $adldap, array $connections = [])
                 new $settings['schema']
             );
 
-            // Try connecting to the provider if `auto_connect` is true.
-            if (isset($settings['auto_connect']) && $settings['auto_connect'] === true) {
-                $provider->connect();
+            if ($this->shouldAutoConnect($settings)) {
+                try {
+                    $provider->connect();
+                } catch (BindException $e) {
+                    // We'll catch and log bind exceptions so
+                    // any connection issues fail gracefully
+                    // in our application.
+                    Log::error($e);
+                }
             }
 
             // Add the provider to the Adldap container.
@@ -129,6 +137,19 @@ protected function newProvider($configuration = [], ConnectionInterface $connect
         return new Provider($configuration, $connection, $schema);
     }
 
+    /**
+     * Determine if the given settings is configured for auto-connecting.
+     *
+     * @param array $settings
+     *
+     * @return bool
+     */
+    protected function shouldAutoConnect(array $settings)
+    {
+        return array_key_exists('auto_connect', $settings)
+            && $settings['auto_connect'] === true;
+    }
+
     /**
      * Determines if the current application is Lumen.
      *