diff options
author | Christian Schrötter <cs@fnx.li> | 2020-07-08 02:43:29 +0300 |
---|---|---|
committer | Christian Schrötter <cs@fnx.li> | 2022-06-28 19:21:05 +0300 |
commit | 2c183656fc1f218c6f983400e229ab2889f7b222 (patch) | |
tree | 8c3b96121e69dba041a6e1bdf6c0829308e69903 | |
parent | 30cc179c0758f7e46d44a4128209da92c4e6060a (diff) |
Implement error check for clickatellcentral provider
Signed-off-by: Christian Schrötter <cs@fnx.li>
-rw-r--r-- | lib/Service/Gateway/SMS/Provider/ClickatellCentral.php | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/lib/Service/Gateway/SMS/Provider/ClickatellCentral.php b/lib/Service/Gateway/SMS/Provider/ClickatellCentral.php index 9cdb47b..2fb72a5 100644 --- a/lib/Service/Gateway/SMS/Provider/ClickatellCentral.php +++ b/lib/Service/Gateway/SMS/Provider/ClickatellCentral.php @@ -53,7 +53,7 @@ class ClickatellCentral implements IProvider { public function send(string $identifier, string $message) { $config = $this->getConfig(); try { - $this->client->get(vsprintf('https://api.clickatell.com/http/sendmsg?user=%s&password=%s&api_id=%u&to=%s&text=%s', [ + $response = $this->client->get(vsprintf('https://api.clickatell.com/http/sendmsg?user=%s&password=%s&api_id=%u&to=%s&text=%s', [ urlencode($config->getUser()), urlencode($config->getPassword()), $config->getApi(), @@ -63,6 +63,10 @@ class ClickatellCentral implements IProvider { } catch (Exception $ex) { throw new SmsTransmissionException(); } + + if ($response->getStatusCode() !== 200 || substr($response->getBody(), 0, 4) !== 'ID: ') { + throw new SmsTransmissionException($response->getBody()); + } } /** |