@@ -87,7 +87,7 @@ discard block |
||
87 | 87 | public function userExists(string $userId): bool |
88 | 88 | { |
89 | 89 | try { |
90 | - $sth = $this->handle->prepare('SELECT userid FROM ' . $this->tableName . ' WHERE userid = ?'); |
|
90 | + $sth = $this->handle->prepare('SELECT userid FROM '.$this->tableName.' WHERE userid = ?'); |
|
91 | 91 | $sth->execute(array($userId)); |
92 | 92 | return (false !== $sth->fetchColumn()); |
93 | 93 | } |
@@ -107,9 +107,9 @@ discard block |
||
107 | 107 | protected function getUserSecret(string $userId): string |
108 | 108 | { |
109 | 109 | try { |
110 | - $sth = $this->handle->prepare('SELECT secret FROM ' . $this->tableName . ' WHERE userid = ?'); |
|
110 | + $sth = $this->handle->prepare('SELECT secret FROM '.$this->tableName.' WHERE userid = ?'); |
|
111 | 111 | $sth->execute(array($userId)); |
112 | - $res=$sth->fetchColumn(); |
|
112 | + $res = $sth->fetchColumn(); |
|
113 | 113 | if ($res === false) { |
114 | 114 | // No result |
115 | 115 | $this->logger->error(sprintf('No result getting secret for user "%s"', $userId)); |
@@ -145,9 +145,9 @@ discard block |
||
145 | 145 | // - The INSERT will fail when displayname has a NOT NULL constraint |
146 | 146 | try { |
147 | 147 | if ($this->userExists($userId)) { |
148 | - $sth = $this->handle->prepare('UPDATE ' . $this->tableName . ' SET secret = ? WHERE userid = ?'); |
|
148 | + $sth = $this->handle->prepare('UPDATE '.$this->tableName.' SET secret = ? WHERE userid = ?'); |
|
149 | 149 | } else { |
150 | - $sth = $this->handle->prepare('INSERT INTO ' . $this->tableName . ' (secret,userid) VALUES (?,?)'); |
|
150 | + $sth = $this->handle->prepare('INSERT INTO '.$this->tableName.' (secret,userid) VALUES (?,?)'); |
|
151 | 151 | } |
152 | 152 | $sth->execute(array($secret, $userId)); |
153 | 153 | } |
@@ -171,7 +171,7 @@ discard block |
||
171 | 171 | $sth->execute(); |
172 | 172 | } |
173 | 173 | catch (Exception $e) { |
174 | - $statusMessage = "UserSecretStorage_PDO error: " . $e->getMessage(); |
|
174 | + $statusMessage = "UserSecretStorage_PDO error: ".$e->getMessage(); |
|
175 | 175 | return false; |
176 | 176 | } |
177 | 177 |
@@ -90,8 +90,7 @@ discard block |
||
90 | 90 | $sth = $this->handle->prepare('SELECT userid FROM ' . $this->tableName . ' WHERE userid = ?'); |
91 | 91 | $sth->execute(array($userId)); |
92 | 92 | return (false !== $sth->fetchColumn()); |
93 | - } |
|
94 | - catch (Exception $e) { |
|
93 | + } catch (Exception $e) { |
|
95 | 94 | $this->logger->error('PDO error checking user exists', array('exception'=>$e, 'userId'=>$userId)); |
96 | 95 | throw ReadWriteException::fromOriginalException($e); |
97 | 96 | } |
@@ -115,8 +114,7 @@ discard block |
||
115 | 114 | $this->logger->error(sprintf('No result getting secret for user "%s"', $userId)); |
116 | 115 | throw new RuntimeException('User not found'); |
117 | 116 | } |
118 | - } |
|
119 | - catch (Exception $e) { |
|
117 | + } catch (Exception $e) { |
|
120 | 118 | $this->logger->error('PDO error getting user', array('exception' => $e, 'userId' => $userId)); |
121 | 119 | throw ReadWriteException::fromOriginalException($e); |
122 | 120 | } |
@@ -150,8 +148,7 @@ discard block |
||
150 | 148 | $sth = $this->handle->prepare('INSERT INTO ' . $this->tableName . ' (secret,userid) VALUES (?,?)'); |
151 | 149 | } |
152 | 150 | $sth->execute(array($secret, $userId)); |
153 | - } |
|
154 | - catch (Exception $e) { |
|
151 | + } catch (Exception $e) { |
|
155 | 152 | $this->logger->error( |
156 | 153 | sprintf('Unable to persist user secret for user "%s" in user secret storage (PDO)', $userId), |
157 | 154 | array('exception'=>$e) |
@@ -169,8 +166,7 @@ discard block |
||
169 | 166 | try { |
170 | 167 | $sth = $this->handle->prepare('SELECT secret FROM '.$this->tableName.' LIMIT 1'); |
171 | 168 | $sth->execute(); |
172 | - } |
|
173 | - catch (Exception $e) { |
|
169 | + } catch (Exception $e) { |
|
174 | 170 | $statusMessage = "UserSecretStorage_PDO error: " . $e->getMessage(); |
175 | 171 | return false; |
176 | 172 | } |
@@ -57,13 +57,13 @@ discard block |
||
57 | 57 | if ($prefix === $this->encryption->get_type()) { |
58 | 58 | // Decrypt the secret if it is prefixed with the current encryption type |
59 | 59 | // Remove the encryption type prefix before decrypting |
60 | - return $this->encryption->decrypt( substr($encryptedSecret, $pos+1) ); |
|
60 | + return $this->encryption->decrypt(substr($encryptedSecret, $pos + 1)); |
|
61 | 61 | } |
62 | 62 | |
63 | 63 | // Check the decryption array for the encryption type to see if there is an encryption |
64 | 64 | // instance defined for it. If so, use that to decrypt the secret. |
65 | 65 | if (isset($this->decryption[$prefix])) { |
66 | - return $this->decryption[$prefix]->decrypt( substr($encryptedSecret, $pos+1) ); |
|
66 | + return $this->decryption[$prefix]->decrypt(substr($encryptedSecret, $pos + 1)); |
|
67 | 67 | } |
68 | 68 | |
69 | 69 | $this->logger->error("Secret for user '$userId' is encrypted with unsupported encryption type '$prefix'"); |
@@ -80,7 +80,7 @@ discard block |
||
80 | 80 | { |
81 | 81 | $encryptedSecret = $this->encryption->encrypt($secret); |
82 | 82 | // Prefix the user secret with the encryption type |
83 | - $this->setUserSecret($userId, $this->encryption->get_type() . ':' . $encryptedSecret); |
|
83 | + $this->setUserSecret($userId, $this->encryption->get_type().':'.$encryptedSecret); |
|
84 | 84 | } |
85 | 85 | |
86 | 86 | /** |
@@ -88,6 +88,6 @@ discard block |
||
88 | 88 | */ |
89 | 89 | public function healthCheck(string &$statusMessage = ''): bool |
90 | 90 | { |
91 | - return true; // Health check is always successful when not implemented |
|
91 | + return true; // Health check is always successful when not implemented |
|
92 | 92 | } |
93 | 93 | } |
@@ -71,6 +71,6 @@ |
||
71 | 71 | */ |
72 | 72 | public function healthCheck(string &$statusMessage = ''): bool |
73 | 73 | { |
74 | - return true; // Health check is always successful when not implemented |
|
74 | + return true; // Health check is always successful when not implemented |
|
75 | 75 | } |
76 | 76 | } |
@@ -51,7 +51,7 @@ discard block |
||
51 | 51 | /** |
52 | 52 | * @throws Tiqr_Message_Exception_SendFailure |
53 | 53 | */ |
54 | - private function getGoogleAccessToken($credentialsFile, $cacheTokens, $tokenCacheDir ) |
|
54 | + private function getGoogleAccessToken($credentialsFile, $cacheTokens, $tokenCacheDir) |
|
55 | 55 | { |
56 | 56 | $client = new Google_Client(); |
57 | 57 | $client->setLogger($this->logger); |
@@ -63,8 +63,8 @@ discard block |
||
63 | 63 | $pool = new FilesystemCachePool($filesystem); |
64 | 64 | |
65 | 65 | //set up a callback to log token refresh |
66 | - $logger=$this->logger; |
|
67 | - $tokenCallback = function ($cacheKey, $accessToken) use ($logger) { |
|
66 | + $logger = $this->logger; |
|
67 | + $tokenCallback = function($cacheKey, $accessToken) use ($logger) { |
|
68 | 68 | $logger->notice(sprintf('New access token received at cache key %s', $cacheKey)); |
69 | 69 | }; |
70 | 70 | $client->setTokenCallback($tokenCallback); |
@@ -97,9 +97,9 @@ discard block |
||
97 | 97 | * @param $retry boolean is this a 2nd attempt |
98 | 98 | * @throws Tiqr_Message_Exception_SendFailure |
99 | 99 | */ |
100 | - private function _sendFirebase(string $deviceToken, string $alert, array $properties, string $projectId, string $credentialsFile, bool $cacheTokens, string $tokenCacheDir, bool $retry=false) |
|
100 | + private function _sendFirebase(string $deviceToken, string $alert, array $properties, string $projectId, string $credentialsFile, bool $cacheTokens, string $tokenCacheDir, bool $retry = false) |
|
101 | 101 | { |
102 | - $apiurl = sprintf('https://fcm.googleapis.com/v1/projects/%s/messages:send',$projectId); |
|
102 | + $apiurl = sprintf('https://fcm.googleapis.com/v1/projects/%s/messages:send', $projectId); |
|
103 | 103 | |
104 | 104 | $fields = [ |
105 | 105 | 'message' => [ |
@@ -113,14 +113,14 @@ discard block |
||
113 | 113 | |
114 | 114 | // Add custom properties |
115 | 115 | foreach ($properties as $k => $v) { |
116 | - $fields['message']['data'][(string)$k] = (string)$v; |
|
116 | + $fields['message']['data'][(string) $k] = (string) $v; |
|
117 | 117 | } |
118 | 118 | // Add message |
119 | 119 | $fields['message']['data']['text'] = $alert; |
120 | 120 | |
121 | 121 | try { |
122 | 122 | $headers = array( |
123 | - 'Authorization: Bearer ' . $this->getGoogleAccessToken($credentialsFile, $cacheTokens, $tokenCacheDir), |
|
123 | + 'Authorization: Bearer '.$this->getGoogleAccessToken($credentialsFile, $cacheTokens, $tokenCacheDir), |
|
124 | 124 | 'Content-Type: application/json', |
125 | 125 | ); |
126 | 126 | } catch (\Google\Exception $e) { |
@@ -147,14 +147,14 @@ discard block |
||
147 | 147 | } |
148 | 148 | |
149 | 149 | if (!empty($errors)) { |
150 | - throw new Tiqr_Message_Exception_SendFailure("Http error occurred: ". $errors, true); |
|
150 | + throw new Tiqr_Message_Exception_SendFailure("Http error occurred: ".$errors, true); |
|
151 | 151 | } |
152 | 152 | |
153 | 153 | // Wait and retry once in case of a 502 Bad Gateway error |
154 | 154 | if ($statusCode === 502 && !($retry)) { |
155 | 155 | $this->logger->warning("Received HTTP 502 Bad Gateway error, retrying once"); |
156 | 156 | sleep(2); |
157 | - $this->_sendFirebase($deviceToken, $alert, $properties, $projectId, $credentialsFile, $cacheTokens, $tokenCacheDir, true); |
|
157 | + $this->_sendFirebase($deviceToken, $alert, $properties, $projectId, $credentialsFile, $cacheTokens, $tokenCacheDir, true); |
|
158 | 158 | return; |
159 | 159 | } |
160 | 160 | |
@@ -165,7 +165,7 @@ discard block |
||
165 | 165 | // handle errors, ignoring registration_id's |
166 | 166 | $response = json_decode($result, true); |
167 | 167 | foreach ($response as $k => $v) { |
168 | - if ($k=="error") { |
|
168 | + if ($k == "error") { |
|
169 | 169 | throw new Tiqr_Message_Exception_SendFailure(sprintf("Error in FCM response: %s", $result), true); |
170 | 170 | } |
171 | 171 | } |