Merge pull request #3 from OsmosysSoftware/revert-1-issue-181

[bookstack] / app / Services / LdapService.php

diff --git a/app/Services/LdapService.php b/app/Services/LdapService.php
index bceed682a96e0657d7bb6225bab77c2c6a320760..71dc9c0e1a57b731048445bcdd78fed57e98c6a2 100644 (file)
--- a/app/Services/LdapService.php
+++ b/app/Services/LdapService.php
@@ -4,10 +4,27 @@
 use BookStack\Exceptions\LdapException;
 use Illuminate\Contracts\Auth\Authenticatable;
 
 use BookStack\Exceptions\LdapException;
 use Illuminate\Contracts\Auth\Authenticatable;
 

+/**
+ * Class LdapService
+ * Handles any app-specific LDAP tasks.
+ * @package BookStack\Services
+ */

 class LdapService
 {
 
 class LdapService
 {
 

+    protected $ldap;

     protected $ldapConnection;
     protected $ldapConnection;

+    protected $config;
+
+    /**
+     * LdapService constructor.
+     * @param Ldap $ldap
+     */
+    public function __construct(Ldap $ldap)
+    {
+        $this->ldap = $ldap;
+        $this->config = config('services.ldap');
+    }

 
     /**
      * Get the details of a user from LDAP using the given username.
 
     /**
      * Get the details of a user from LDAP using the given username.


@@ -19,19 +36,21 @@ class LdapService
     public function getUserDetails($userName)
     {
         $ldapConnection = $this->getConnection();
     public function getUserDetails($userName)
     {
         $ldapConnection = $this->getConnection();

+        $this->bindSystemUser($ldapConnection);

 
         // Find user
 
         // Find user

-        $userFilter = $this->buildFilter(config('services.ldap.user_filter'), ['user' => $userName]);
-        $baseDn = config('services.ldap.base_dn');
-        $ldapSearch = ldap_search($ldapConnection, $baseDn, $userFilter, ['cn', 'uid', 'dn']);
-        $users = ldap_get_entries($ldapConnection, $ldapSearch);
+        $userFilter = $this->buildFilter($this->config['user_filter'], ['user' => $userName]);
+        $baseDn = $this->config['base_dn'];
+        $emailAttr = $this->config['email_attribute'];
+        $users = $this->ldap->searchAndGetEntries($ldapConnection, $baseDn, $userFilter, ['cn', 'uid', 'dn', $emailAttr]);

         if ($users['count'] === 0) return null;
 
         $user = $users[0];
         return [
         if ($users['count'] === 0) return null;
 
         $user = $users[0];
         return [

-            'uid'  => $user['uid'][0],
-            'name' => $user['cn'][0],
-            'dn'   => $user['dn']
+            'uid'   => (isset($user['uid'])) ? $user['uid'][0] : $user['dn'],
+            'name'  => $user['cn'][0],
+            'dn'    => $user['dn'],
+            'email' => (isset($user[$emailAttr])) ? (is_array($user[$emailAttr]) ? $user[$emailAttr][0] : $user[$emailAttr]) : null

         ];
     }
 
         ];
     }
 


@@ -49,7 +68,12 @@ class LdapService
         if ($ldapUser['uid'] !== $user->external_auth_id) return false;
 
         $ldapConnection = $this->getConnection();
         if ($ldapUser['uid'] !== $user->external_auth_id) return false;
 
         $ldapConnection = $this->getConnection();

-        $ldapBind = @ldap_bind($ldapConnection, $ldapUser['dn'], $password);
+        try {
+            $ldapBind = $this->ldap->bind($ldapConnection, $ldapUser['dn'], $password);
+        } catch (\ErrorException $e) {
+            $ldapBind = false;
+        }
+

         return $ldapBind;
     }
 
         return $ldapBind;
     }
 


@@ -61,17 +85,17 @@ class LdapService
      */
     protected function bindSystemUser($connection)
     {
      */
     protected function bindSystemUser($connection)
     {

-        $ldapDn = config('services.ldap.dn');
-        $ldapPass = config('services.ldap.pass');
+        $ldapDn = $this->config['dn'];
+        $ldapPass = $this->config['pass'];

 
         $isAnonymous = ($ldapDn === false || $ldapPass === false);
         if ($isAnonymous) {
 
         $isAnonymous = ($ldapDn === false || $ldapPass === false);
         if ($isAnonymous) {

-            $ldapBind = ldap_bind($connection);
+            $ldapBind = $this->ldap->bind($connection);

         } else {
         } else {

-            $ldapBind = ldap_bind($connection, $ldapDn, $ldapPass);
+            $ldapBind = $this->ldap->bind($connection, $ldapDn, $ldapPass);

         }
 
         }
 

-        if (!$ldapBind) throw new LdapException('LDAP access failed using ' . $isAnonymous ? ' anonymous bind.' : ' given dn & pass details');
+        if (!$ldapBind) throw new LdapException(($isAnonymous ? trans('errors.ldap_fail_anonymous') : trans('errors.ldap_fail_authed')));

     }
 
     /**
     }
 
     /**


@@ -85,20 +109,26 @@ class LdapService
         if ($this->ldapConnection !== null) return $this->ldapConnection;
 
         // Check LDAP extension in installed
         if ($this->ldapConnection !== null) return $this->ldapConnection;
 
         // Check LDAP extension in installed

-        if (!function_exists('ldap_connect')) {
-            throw new LdapException('LDAP PHP extension not installed');
+        if (!function_exists('ldap_connect') && config('app.env') !== 'testing') {
+            throw new LdapException(trans('errors.ldap_extension_not_installed'));

         }
 
         }
 

-        // Get port from server string if specified.
-        $ldapServer = explode(':', config('services.ldap.server'));
-        $ldapConnection = ldap_connect($ldapServer[0], count($ldapServer) > 1 ? $ldapServer[1] : 389);
+        // Get port from server string and protocol if specified.
+        $ldapServer = explode(':', $this->config['server']);
+        $hasProtocol = preg_match('/^ldaps{0,1}\:\/\//', $this->config['server']) === 1;
+        if (!$hasProtocol) array_unshift($ldapServer, '');
+        $hostName = $ldapServer[0] . ($hasProtocol?':':'') . $ldapServer[1];
+        $defaultPort = $ldapServer[0] === 'ldaps' ? 636 : 389;
+        $ldapConnection = $this->ldap->connect($hostName, count($ldapServer) > 2 ? intval($ldapServer[2]) : $defaultPort);

 
         if ($ldapConnection === false) {
 
         if ($ldapConnection === false) {

-            throw new LdapException('Cannot connect to ldap server, Initial connection failed');
+            throw new LdapException(trans('errors.ldap_cannot_connect'));

         }
 
         // Set any required options
         }
 
         // Set any required options

-        ldap_set_option($ldapConnection, LDAP_OPT_PROTOCOL_VERSION, 3); // TODO - make configurable
+        if ($this->config['version']) {
+            $this->ldap->setVersion($ldapConnection, $this->config['version']);
+        }

 
         $this->ldapConnection = $ldapConnection;
         return $this->ldapConnection;
 
         $this->ldapConnection = $ldapConnection;
         return $this->ldapConnection;


@@ -106,7 +136,7 @@ class LdapService
 
     /**
      * Build a filter string by injecting common variables.
 
     /**
      * Build a filter string by injecting common variables.

-     * @param       $filterString
+     * @param string $filterString

      * @param array $attrs
      * @return string
      */
      * @param array $attrs
      * @return string
      */