simplesamlphp / simplesamlphp-module-sqlauth

lib/Auth/Source/SQL.php

Spacing +17 added lines, -17 removed lines

@@ -68,14 +68,14 @@ 
         // Make sure that all required parameters are present.
         foreach (['dsn', 'username', 'password', 'query'] as $param) {
             if (!array_key_exists($param, $config)) {
-                throw new Exception('Missing required attribute \'' . $param .
-                    '\' for authentication source ' . $this->authId);
+                throw new Exception('Missing required attribute \''.$param.
+                    '\' for authentication source '.$this->authId);
             }
 
             if (!is_string($config[$param])) {
-                throw new Exception('Expected parameter \'' . $param .
-                    '\' for authentication source ' . $this->authId .
-                    ' to be a string. Instead it was: ' .
+                throw new Exception('Expected parameter \''.$param.
+                    '\' for authentication source '.$this->authId.
+                    ' to be a string. Instead it was: '.
                     var_export($config[$param], true));
             }
         }
@@ -101,10 +101,10 @@ 
             $db = new PDO($this->dsn, $this->username, $this->password, $this->options);
         } catch (PDOException $e) {
             // Obfuscate the password if it's part of the dsn
-            $obfuscated_dsn =  preg_replace('/(user|password)=(.*?([;]|$))/', '${1}=***', $this->dsn);
+            $obfuscated_dsn = preg_replace('/(user|password)=(.*?([;]|$))/', '${1}=***', $this->dsn);
 
-            throw new \Exception('sqlauth:' . $this->authId . ': - Failed to connect to \'' .
-                $obfuscated_dsn . '\': ' . $e->getMessage());
+            throw new \Exception('sqlauth:'.$this->authId.': - Failed to connect to \''.
+                $obfuscated_dsn.'\': '.$e->getMessage());
         }
 
         $db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
@@ -148,30 +148,30 @@ 
         try {
             $sth = $db->prepare($this->query);
         } catch (PDOException $e) {
-            throw new Exception('sqlauth:' . $this->authId .
-                ': - Failed to prepare query: ' . $e->getMessage());
+            throw new Exception('sqlauth:'.$this->authId.
+                ': - Failed to prepare query: '.$e->getMessage());
         }
 
         try {
             $sth->execute(['username' => $username, 'password' => $password]);
         } catch (PDOException $e) {
-            throw new Exception('sqlauth:' . $this->authId .
-                ': - Failed to execute query: ' . $e->getMessage());
+            throw new Exception('sqlauth:'.$this->authId.
+                ': - Failed to execute query: '.$e->getMessage());
         }
 
         try {
             $data = $sth->fetchAll(PDO::FETCH_ASSOC);
         } catch (PDOException $e) {
-            throw new Exception('sqlauth:' . $this->authId .
-                ': - Failed to fetch result set: ' . $e->getMessage());
+            throw new Exception('sqlauth:'.$this->authId.
+                ': - Failed to fetch result set: '.$e->getMessage());
         }
 
-        Logger::info('sqlauth:' . $this->authId . ': Got ' . count($data) .
+        Logger::info('sqlauth:'.$this->authId.': Got '.count($data).
             ' rows from database');
 
         if (count($data) === 0) {
             // No rows returned - invalid username/password
-            Logger::error('sqlauth:' . $this->authId .
+            Logger::error('sqlauth:'.$this->authId.
                 ': No rows in result set. Probably wrong username/password.');
             throw new Error\Error('WRONGUSERPASS');
         }
@@ -202,7 +202,7 @@ 
             }
         }
 
-        Logger::info('sqlauth:' . $this->authId . ': Attributes: ' . implode(',', array_keys($attributes)));
+        Logger::info('sqlauth:'.$this->authId.': Attributes: '.implode(',', array_keys($attributes)));
 
         return $attributes;
     }