@@ -147,8 +147,7 @@ discard block |
||
147 | 147 | if (!empty($secure)) |
148 | 148 | { |
149 | 149 | return 'on' == strtolower($secure) || '1' == $secure; |
150 | - } |
|
151 | - else if ('443' == $this->request->server('SERVER_PORT')) |
|
150 | + } else if ('443' == $this->request->server('SERVER_PORT')) |
|
152 | 151 | { |
153 | 152 | return true; |
154 | 153 | } |
@@ -236,10 +235,12 @@ discard block |
||
236 | 235 | |
237 | 236 | if (property_exists($response, 'errorCode')) |
238 | 237 | { |
239 | - if ($response->errorCode == 4) // errorCode 4 means that this device wasn't registered |
|
238 | + if ($response->errorCode == 4) { |
|
239 | + // errorCode 4 means that this device wasn't registered |
|
240 | 240 | { |
241 | 241 | throw new AccessDeniedHttpException($this->user->lang('TFA_NOT_REGISTERED')); |
242 | 242 | } |
243 | + } |
|
243 | 244 | throw new BadRequestHttpException($this->user->lang('TFA_SOMETHING_WENT_WRONG')); |
244 | 245 | } |
245 | 246 | $result = new AuthenticationResponse($response->signatureData, $response->clientData, $response->keyHandle, $response->errorCode); |
@@ -253,12 +254,10 @@ discard block |
||
253 | 254 | |
254 | 255 | $sql = 'UPDATE ' . $this->registration_table . ' SET ' . $this->db->sql_build_array('UPDATE', $sql_ary) . ' WHERE registration_id = ' . (int) $reg->getId(); |
255 | 256 | $this->db->sql_query($sql); |
256 | - } |
|
257 | - catch (U2fError $error) |
|
257 | + } catch (U2fError $error) |
|
258 | 258 | { |
259 | 259 | $this->createError($error); |
260 | - } |
|
261 | - catch (\InvalidArgumentException $invalid) |
|
260 | + } catch (\InvalidArgumentException $invalid) |
|
262 | 261 | { |
263 | 262 | throw new BadRequestHttpException($this->user->lang('TFA_SOMETHING_WENT_WRONG') . '<br />' . $invalid->getMessage(), $invalid); |
264 | 263 | } |
@@ -295,8 +294,7 @@ discard block |
||
295 | 294 | if ($count == 0) |
296 | 295 | { |
297 | 296 | trigger_error('TFA_UNABLE_TO_UPDATE_SESSION'); |
298 | - } |
|
299 | - else if ($count > 1) |
|
297 | + } else if ($count > 1) |
|
300 | 298 | { |
301 | 299 | // Reset sessions table. We had multiple sessions with same ID!!! |
302 | 300 | $sql_ary['u2f_request'] = ''; |
@@ -340,8 +338,7 @@ discard block |
||
340 | 338 | ); |
341 | 339 | |
342 | 340 | $this->update_session($sql_ary); |
343 | - } |
|
344 | - catch (U2fError $err) |
|
341 | + } catch (U2fError $err) |
|
345 | 342 | { |
346 | 343 | $this->createError($err); |
347 | 344 | } |