Passed
Push — master ( 14ffee...a9ddb2 )
by Tim
02:32
created
lib/Auth/Process/OTP.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -152,7 +152,7 @@  discard block
 block discarded – undo
152 152
         // check for previous auth
153 153
         if (!is_null($key_id) && in_array($key_id, $attrs[$this->keyIdAttr], true)) {
154 154
             // we were already authenticated using a valid yubikey
155
-            Logger::info('Reusing previous YubiKey authentication with key "' . $key_id . '".');
155
+            Logger::info('Reusing previous YubiKey authentication with key "'.$key_id.'".');
156 156
             return;
157 157
         }
158 158
 
@@ -205,7 +205,7 @@  discard block
 block discarded – undo
205 205
 
206 206
         // obtain the identity of the yubikey
207 207
         $kid = substr($otp, 0, -32);
208
-        Logger::debug('Verifying Yubikey ID "' . $kid . '"');
208
+        Logger::debug('Verifying Yubikey ID "'.$kid.'"');
209 209
 
210 210
         // verify the OTP against the API
211 211
         $api = new \Yubikey\Validate($cfg['apiKey'], $cfg['apiClient']);
@@ -214,7 +214,7 @@  discard block
 block discarded – undo
214 214
 
215 215
         // verify the identity corresponds to this user
216 216
         if (!in_array($kid, $cfg['keyIDs'], true)) {
217
-            Logger::warning('The YubiKey "' . $kid . '" is not valid for this user.');
217
+            Logger::warning('The YubiKey "'.$kid.'" is not valid for this user.');
218 218
             Logger::stats('yubikey:otp: invalid YubiKey.');
219 219
             return false;
220 220
         }
@@ -232,10 +232,10 @@  discard block
 block discarded – undo
232 232
                 $cfg['self'],
233 233
                 'logoutHandler'
234 234
             );
235
-            Logger::info('Successful authentication with YubiKey "' . $kid . '".');
235
+            Logger::info('Successful authentication with YubiKey "'.$kid.'".');
236 236
             return true;
237 237
         }
238
-        Logger::warning('Couldn\'t successfully authenticate YubiKey "' . $kid . '".');
238
+        Logger::warning('Couldn\'t successfully authenticate YubiKey "'.$kid.'".');
239 239
         return false;
240 240
     }
241 241
 
@@ -249,7 +249,7 @@  discard block
 block discarded – undo
249 249
     {
250 250
         $session = \SimpleSAML\Session::getSessionFromRequest();
251 251
         $keyid = $session->getData('yubikey:auth', $this->authid);
252
-        Logger::info('Removing valid YubiKey authentication with key "' . $keyid . '".');
252
+        Logger::info('Removing valid YubiKey authentication with key "'.$keyid.'".');
253 253
         $session->deleteData('yubikey:auth', $this->authid);
254 254
     }
255 255
 }
Please login to merge, or discard this patch.