Merge master->develop for 1.0.0 release

Author: mheap

.travis.yml

@@ -2,8 +2,10 @@ language: php
 php:
   - '5.6'
   - '7.0'
+  - '7.1'
+  - '7.2'
 
 before_script:
   - composer install
 
-script: ./vendor/bin/phpunit -v --configuration ./phpunit.xml.dist
+script: ./vendor/bin/phpunit -v --configuration ./phpunit.xml.dist

CHANGELOG.md

@@ -0,0 +1,7 @@
+This is a high level list of changes for each version of the Nexmo PHP client. 
+
+### 1.0.0 (Not released yet) [Show Diff](https://github.com/Nexmo/nexmo-php/compare/develop)
+
+* Renamed `Nexmo\Client\Credentials\SharedSecret` to `Nexmo\Client\Credentials\SignatureSecret` [#39](https://github.com/Nexmo/nexmo-php/pull/39)
+* Added ability to set callback URL via `setCallback()` [#34](https://github.com/Nexmo/nexmo-php/pull/34)
+* Rate limit error when searching messages [#37](https://github.com/Nexmo/nexmo-php/pull/37)

README.md

@@ -2,6 +2,8 @@ Nexmo Client Library for PHP
 ============================
 [![Build Status](https://api.travis-ci.org/Nexmo/nexmo-php.svg?branch=master)](https://travis-ci.org/Nexmo/nexmo-php)
 
+*This library requires a minimum PHP version of 5.6*
+
 This is the PHP client library for use Nexmo's API. To use this, you'll need a Nexmo account. Sign up [for free at 
 nexmo.com][signup]. This is currently a beta release, see [contributing](#contributing) for more information.
 

composer.json

@@ -15,7 +15,7 @@
     "email": "devrel@nexmo.com"
   },
   "require": {
-    "php": ">=5.4",
+    "php": ">=5.6",
     "php-http/client-implementation": "^1.0",
     "zendframework/zend-diactoros": "^1.3",
     "php-http/guzzle6-adapter": "^1.0",

src/Call/Collection.php

@@ -144,10 +144,25 @@ protected function getException(ResponseInterface $response)
         $body = json_decode($response->getBody()->getContents(), true);
         $status = $response->getStatusCode();
 
+        // Error responses aren't consistent. Some are generated within the
+        // proxy and some are generated within voice itself. This handles
+        // both cases
+
+        // This message isn't very useful, but we shouldn't ever see it
+        $errorTitle = 'Unexpected error';
+
+        if (isset($body['title'])) {
+            $errorTitle = $body['title'];
+        }
+
+        if (isset($body['error_title'])) {
+            $errorTitle = $body['error_title'];
+        }
+
         if($status >= 400 AND $status < 500) {
-            $e = new Exception\Request($body['error_title'], $status);
+            $e = new Exception\Request($errorTitle, $status);
         } elseif($status >= 500 AND $status < 600) {
-            $e = new Exception\Server($body['error_title'], $status);
+            $e = new Exception\Server($errorTitle, $status);
         } else {
             $e = new Exception\Exception('Unexpected HTTP Status Code');
             throw $e;

src/Client.php

@@ -13,7 +13,7 @@
 use Nexmo\Client\Credentials\CredentialsInterface;
 use Nexmo\Client\Credentials\Keypair;
 use Nexmo\Client\Credentials\OAuth;
-use Nexmo\Client\Credentials\SharedSecret;
+use Nexmo\Client\Credentials\SignatureSecret;
 use Nexmo\Client\Factory\FactoryInterface;
 use Nexmo\Client\Factory\MapFactory;
 use Nexmo\Client\Response\Response;
@@ -22,6 +22,7 @@
 use Nexmo\Verify\Verification;
 use Psr\Http\Message\RequestInterface;
 use Zend\Diactoros\Uri;
+use Zend\Diactoros\Request;
 
 /**
  * Nexmo API Client, allows access to the API from PHP.
@@ -76,7 +77,7 @@ public function __construct(CredentialsInterface $credentials, $options = array(
         $this->setHttpClient($client);
 
         //make sure we know how to use the credentials
-        if(!($credentials instanceof Container) && !($credentials instanceof Basic) && !($credentials instanceof SharedSecret) && !($credentials instanceof OAuth)){
+        if(!($credentials instanceof Container) && !($credentials instanceof Basic) && !($credentials instanceof SignatureSecret) && !($credentials instanceof OAuth) && !($credentials instanceof Keypair)){
             throw new \RuntimeException('unknown credentials type: ' . get_class($credentials));
         }
 
@@ -135,7 +136,7 @@ public function setFactory(FactoryInterface $factory)
      * @param Signature $signature
      * @return RequestInterface
      */
-    public static function signRequest(RequestInterface $request, SharedSecret $credentials)
+    public static function signRequest(RequestInterface $request, SignatureSecret $credentials)
     {
         switch($request->getHeaderLine('content-type')){
             case 'application/json':
@@ -144,7 +145,7 @@ public static function signRequest(RequestInterface $request, SharedSecret $cred
                 $content = $body->getContents();
                 $params = json_decode($content, true);
                 $params['api_key'] = $credentials['api_key'];
-                $signature = new Signature($params, $credentials['shared_secret']);
+                $signature = new Signature($params, $credentials['signature_secret']);
                 $body->rewind();
                 $body->write(json_encode($signature->getSignedParams()));
                 break;
@@ -155,7 +156,7 @@ public static function signRequest(RequestInterface $request, SharedSecret $cred
                 $params = [];
                 parse_str($content, $params);
                 $params['api_key'] = $credentials['api_key'];
-                $signature = new Signature($params, $credentials['shared_secret']);
+                $signature = new Signature($params, $credentials['signature_secret']);
                 $params = $signature->getSignedParams();
                 $body->rewind();
                 $body->write(http_build_query($params, null, '&'));
@@ -164,7 +165,7 @@ public static function signRequest(RequestInterface $request, SharedSecret $cred
                 $query = [];
                 parse_str($request->getUri()->getQuery(), $query);
                 $query['api_key'] = $credentials['api_key'];
-                $signature = new Signature($query, $credentials['shared_secret']);
+                $signature = new Signature($query, $credentials['signature_secret']);
                 $request = $request->withUri($request->getUri()->withQuery(http_build_query($signature->getSignedParams())));
                 break;
         }
@@ -206,6 +207,83 @@ public static function authRequest(RequestInterface $request, Basic $credentials
     }
 
     /**
+     * Takes a URL and a key=>value array to generate a GET PSR-7 request object
+     *
+     * @param string $url The URL to make a request to
+     * @param array $params Key=>Value array of data to use as the query string
+     * @return \Psr\Http\Message\ResponseInterface
+     */
+    public function get($url, array $params = [])
+    {
+       $queryString = '?' . http_build_query($params);
+
+       $url = $url . $queryString;
+
+       $request = new Request(
+            $url,
+            'GET'
+        );
+
+        return $this->send($request);
+    }
+
+    /**
+     * Takes a URL and a key=>value array to generate a POST PSR-7 request object
+     *
+     * @param string $url The URL to make a request to
+     * @param array $params Key=>Value array of data to send
+     * @return \Psr\Http\Message\ResponseInterface
+     */
+    public function post($url, array $params)
+    {
+        $request = new Request(
+            $url,
+            'POST',
+            'php://temp',
+            ['content-type' => 'application/json']
+        );
+
+        $request->getBody()->write(json_encode($params));
+        return $this->send($request);
+    }
+
+    /**
+     * Takes a URL and a key=>value array to generate a PUT PSR-7 request object
+     *
+     * @param string $url The URL to make a request to
+     * @param array $params Key=>Value array of data to send
+     * @return \Psr\Http\Message\ResponseInterface
+     */
+    public function put($url, array $params)
+    {
+        $request = new Request(
+            $url,
+            'PUT',
+            'php://temp',
+            ['content-type' => 'application/json']
+        );
+
+        $request->getBody()->write(json_encode($params));
+        return $this->send($request);
+    }
+
+    /**
+     * Takes a URL and a key=>value array to generate a DELETE PSR-7 request object
+     *
+     * @param string $url The URL to make a request to
+     * @return \Psr\Http\Message\ResponseInterface
+     */
+    public function delete($url)
+    {
+        $request = new Request(
+            $url,
+            'DELETE'
+        );
+
+        return $this->send($request);
+    }
+
+     /**
      * Wraps the HTTP Client, creates a new PSR-7 request adding authentication, signatures, etc.
      *
      * @param \Psr\Http\Message\RequestInterface $request
@@ -220,8 +298,8 @@ public function send(\Psr\Http\Message\RequestInterface $request)
                 $request = self::authRequest($request, $this->credentials->get(Basic::class));
             }
         } elseif($this->credentials instanceof Keypair){
-            $request = $request->withHeader('Authorization', 'Bearer ' . $this->credentials->get(Keypair::class)->generateJwt());
-        } elseif($this->credentials instanceof SharedSecret){
+            $request = $request->withHeader('Authorization', 'Bearer ' . $this->credentials->generateJwt());
+        } elseif($this->credentials instanceof SignatureSecret){
             $request = self::signRequest($request, $this->credentials);
         } elseif($this->credentials instanceof Basic){
             $request = self::authRequest($request, $this->credentials);

src/Client/Credentials/Container.php

@@ -12,7 +12,7 @@ class Container extends AbstractCredentials implements CredentialsInterface
 {
     protected $types = [
         Basic::class,
-        SharedSecret::class,
+        SignatureSecret::class,
         Keypair::class
     ];
 

src/Client/Credentials/SharedSecret.php …c/Client/Credentials/SignatureSecret.phpsrc/Client/Credentials/SharedSecret.php renamed to src/Client/Credentials/SignatureSecret.php src/Client/Credentials/SharedSecret.php renamed to src/Client/Credentials/SignatureSecret.php

@@ -8,17 +8,17 @@
 
 namespace Nexmo\Client\Credentials;
 
-class SharedSecret extends AbstractCredentials implements CredentialsInterface
+class SignatureSecret extends AbstractCredentials implements CredentialsInterface
 {
     /**
-     * Create a credential set with an API key and shared secret.
+     * Create a credential set with an API key and signature secret.
      *
      * @param string $key    API Key
-     * @param string $secret Shared Secret
+     * @param string $signature_secret Signature Secret
      */
-    public function __construct($key, $secret)
+    public function __construct($key, $signature_secret)
     {
         $this->credentials['api_key'] = $key;
-        $this->credentials['shared_secret'] = $secret;
+        $this->credentials['signature_secret'] = $signature_secret;
     }
-}
+}

src/Entity/RequestArrayTrait.php

@@ -40,6 +40,13 @@ public function getRequestData($sent = true)
             return $query;
         }
 
+        // Trigger a pre-getRequestData() hook for any last minute
+        // decision making that needs to be done, but only if
+        // it hasn't been sent already
+        if (method_exists($this, 'preGetRequestDataHook')) {
+            $this->preGetRequestDataHook();
+        }
+
         return $this->requestData;
     }    
 

src/Message/AutoDetect.php

@@ -0,0 +1,38 @@
+<?php
+/**
+ * Nexmo Client Library for PHP
+ *
+ * @copyright Copyright (c) 2016 Nexmo, Inc. (http://nexmo.com)
+ * @license   https://github.com/Nexmo/nexmo-php/blob/master/LICENSE.txt MIT License
+ */
+
+namespace Nexmo\Message;
+
+/**
+ * SMS Text Message
+ */
+class AutoDetect extends Message
+{
+    const TYPE = 'text';
+    
+    /**
+     * Message Body
+     * @var string
+     */
+    protected $text;
+    
+    /**
+     * Create a new SMS text message.
+     * 
+     * @param string $to
+     * @param string $from
+     * @param string $text
+     * @param array  $additional
+     */
+    public function __construct($to, $from, $text, $additional = [])
+    {
+        parent::__construct($to, $from, $additional);
+        $this->enableEncodingDetection();
+        $this->requestData['text'] = (string) $text;
+    }
+}