Also use RequestInterface in Signature Methods.
This commit is contained in:
		| @@ -46,13 +46,13 @@ public function getName() | |||||||
|      * Please note that the second encoding MUST NOT happen in the SignatureMethod, as |      * Please note that the second encoding MUST NOT happen in the SignatureMethod, as | ||||||
|      * OAuthRequest handles this! |      * OAuthRequest handles this! | ||||||
|      * |      * | ||||||
|      * @param JacobKiers\OAuth\Request $request |      * @param JacobKiers\OAuth\RequestInterface $request | ||||||
|      * @param JacobKiers\OAuth\Client  $client |      * @param JacobKiers\OAuth\Client  $client | ||||||
|      * @param JacobKiers\OAuth\Token   $token |      * @param JacobKiers\OAuth\Token   $token | ||||||
|      * |      * | ||||||
|      * @return string |      * @return string | ||||||
|      */ |      */ | ||||||
|     public function buildSignature(Request $request, Client $client, Token $token = null) |     public function buildSignature(RequestInterface $request, Client $client, Token $token = null) | ||||||
|     { |     { | ||||||
|         $base_string = $request->getOAuthSignatureBaseString(); |         $base_string = $request->getOAuthSignatureBaseString(); | ||||||
|         $key = $this->getSignatureKey($client, $token); |         $key = $this->getSignatureKey($client, $token); | ||||||
|   | |||||||
| @@ -44,13 +44,13 @@ public function getName() | |||||||
|      * Please note that the second encoding MUST NOT happen in the SignatureMethod, as |      * Please note that the second encoding MUST NOT happen in the SignatureMethod, as | ||||||
|      * OAuthRequest handles this! |      * OAuthRequest handles this! | ||||||
|      * |      * | ||||||
|      * @param JacobKiers\OAuth\Request $request |      * @param JacobKiers\OAuth\RequestInterface $request | ||||||
|      * @param JacobKiers\OAuth\Client  $client |      * @param JacobKiers\OAuth\Client  $client | ||||||
|      * @param JacobKiers\OAuth\Token   $token |      * @param JacobKiers\OAuth\Token   $token | ||||||
|      * |      * | ||||||
|      * @return string |      * @return string | ||||||
|      */ |      */ | ||||||
|     public function buildSignature(Request $request, Client $client, Token $token = null) |     public function buildSignature(RequestInterface $request, Client $client, Token $token = null) | ||||||
|     { |     { | ||||||
|         return $this->getSignatureKey($client, $token); |         return $this->getSignatureKey($client, $token); | ||||||
|     } |     } | ||||||
|   | |||||||
| @@ -79,7 +79,7 @@ public function __construct($http_method, $http_url, array $parameters = null) | |||||||
|      * @param string $http_url    Request URL. |      * @param string $http_url    Request URL. | ||||||
|      * @param array  $parameters  HTTP parameters. |      * @param array  $parameters  HTTP parameters. | ||||||
|      * |      * | ||||||
|      * @return JacobKiers\OAuth\Request |      * @return JacobKiers\OAuth\RequestInterface | ||||||
|      */ |      */ | ||||||
|     public static function fromRequest($http_method = null, $http_url = null, $parameters = null) |     public static function fromRequest($http_method = null, $http_url = null, $parameters = null) | ||||||
|     { |     { | ||||||
| @@ -137,7 +137,7 @@ public static function fromRequest($http_method = null, $http_url = null, $param | |||||||
|      * @param string                 $http_url |      * @param string                 $http_url | ||||||
|      * @param array                  $parameters |      * @param array                  $parameters | ||||||
|      * |      * | ||||||
|      * @return JacobKiers\OAuth\Request |      * @return JacobKiers\OAuth\RequestInterface | ||||||
|      */ |      */ | ||||||
|     public static function fromClientAndToken( |     public static function fromClientAndToken( | ||||||
|         Client $client, |         Client $client, | ||||||
|   | |||||||
| @@ -58,13 +58,13 @@ abstract protected function fetchPrivateCert(&$request); | |||||||
|     /** |     /** | ||||||
|      * Build up the signature. |      * Build up the signature. | ||||||
|      * |      * | ||||||
|      * @param JacobKiers\OAuth\Request $request |      * @param JacobKiers\OAuth\RequestInterface $request | ||||||
|      * @param JacobKiers\OAuth\Client  $client |      * @param JacobKiers\OAuth\Client  $client | ||||||
|      * @param JacobKiers\OAuth\Token   $token |      * @param JacobKiers\OAuth\Token   $token | ||||||
|      * |      * | ||||||
|      * @return string |      * @return string | ||||||
|      */ |      */ | ||||||
|     public function buildSignature(Request $request, Client $client, Token $token = null) |     public function buildSignature(RequestInterface $request, Client $client, Token $token = null) | ||||||
|     { |     { | ||||||
|         $base_string = $request->getOAuthSignatureBaseString(); |         $base_string = $request->getOAuthSignatureBaseString(); | ||||||
|  |  | ||||||
| @@ -86,14 +86,14 @@ public function buildSignature(Request $request, Client $client, Token $token = | |||||||
|     /** |     /** | ||||||
|      * Verifies that a given signature is correct. |      * Verifies that a given signature is correct. | ||||||
|      * |      * | ||||||
|      * @param JacobKiers\OAuth\Request  $request |      * @param JacobKiers\OAuth\RequestInterface  $request | ||||||
|      * @param JacobKiers\OAuth\Consumer $client |      * @param JacobKiers\OAuth\Consumer $client | ||||||
|      * @param JacobKiers\OAuth\Token    $token |      * @param JacobKiers\OAuth\Token    $token | ||||||
|      * @param string                   $signature |      * @param string                   $signature | ||||||
|      * |      * | ||||||
|      * @return bool |      * @return bool | ||||||
|      */ |      */ | ||||||
|     public function checkSignature(Request $request, Client $client, Token $token, $signature) |     public function checkSignature(RequestInterface $request, Client $client, Token $token, $signature) | ||||||
|     { |     { | ||||||
|         $base_string = $request->getOAuthSignatureBaseString(); |         $base_string = $request->getOAuthSignatureBaseString(); | ||||||
|  |  | ||||||
|   | |||||||
| @@ -77,7 +77,7 @@ public function addSignatureMethod(SignatureMethod $signature_method) | |||||||
|      * |      * | ||||||
|      * Returns the request token on success |      * Returns the request token on success | ||||||
|      * |      * | ||||||
|      * @param JacobKiers\OAuth\RequestInterface $request |      * @param JacobKiers\OAuth\RequestInterfaceInterface $request | ||||||
|      * |      * | ||||||
|      * @return JacobKiers\OAuth\Token |      * @return JacobKiers\OAuth\Token | ||||||
|      */ |      */ | ||||||
| @@ -103,7 +103,7 @@ public function fetchRequestToken(RequestInterface &$request) | |||||||
|      * |      * | ||||||
|      * Returns the access token on success. |      * Returns the access token on success. | ||||||
|      * |      * | ||||||
|      * @param JacobKiers\OAuth\RequestInterface $request |      * @param JacobKiers\OAuth\RequestInterfaceInterface $request | ||||||
|      * |      * | ||||||
|      * @return JacobKiers\OAuth\Token |      * @return JacobKiers\OAuth\Token | ||||||
|      */ |      */ | ||||||
| @@ -127,7 +127,7 @@ public function fetchAccessToken(RequestInterface &$request) | |||||||
|     /** |     /** | ||||||
|      * Verify an api call, checks all the parameters. |      * Verify an api call, checks all the parameters. | ||||||
|      * |      * | ||||||
|      * @param JacobKiers\OAuth\RequestInterface $request |      * @param JacobKiers\OAuth\RequestInterfaceInterface $request | ||||||
|      * |      * | ||||||
|      * @return array Client and Token |      * @return array Client and Token | ||||||
|      */ |      */ | ||||||
| @@ -145,7 +145,7 @@ public function verifyRequest(RequestInterface &$request) | |||||||
|     /** |     /** | ||||||
|      * Check that version is 1.0. |      * Check that version is 1.0. | ||||||
|      * |      * | ||||||
|      * @param JacobKiers\OAuth\RequestInterface $request |      * @param JacobKiers\OAuth\RequestInterfaceInterface $request | ||||||
|      * |      * | ||||||
|      * @return string |      * @return string | ||||||
|      * |      * | ||||||
| @@ -168,7 +168,7 @@ private function getVersion(RequestInterface &$request) | |||||||
|     /** |     /** | ||||||
|      * Get the signature method name, and if it is supported. |      * Get the signature method name, and if it is supported. | ||||||
|      * |      * | ||||||
|      * @param JacobKiers\OAuth\RequestInterface $request |      * @param JacobKiers\OAuth\RequestInterfaceInterface $request | ||||||
|      * |      * | ||||||
|      * @return string Signature method name. |      * @return string Signature method name. | ||||||
|      * |      * | ||||||
| @@ -196,7 +196,7 @@ private function getSignatureMethod(RequestInterface $request) | |||||||
|     /** |     /** | ||||||
|      * Try to find the client for the provided request's client key. |      * Try to find the client for the provided request's client key. | ||||||
|      * |      * | ||||||
|      * @param JacobKiers\OAuth\RequestInterface $request |      * @param JacobKiers\OAuth\RequestInterfaceInterface $request | ||||||
|      * |      * | ||||||
|      * @return JacobKiers\OAuth\Client |      * @return JacobKiers\OAuth\Client | ||||||
|      * |      * | ||||||
| @@ -221,7 +221,7 @@ private function getClient(RequestInterface $request) | |||||||
|     /** |     /** | ||||||
|      * Try to find the token for the provided request's token key. |      * Try to find the token for the provided request's token key. | ||||||
|      * |      * | ||||||
|      * @param JacobKiers\OAuth\RequestInterface $request |      * @param JacobKiers\OAuth\RequestInterfaceInterface $request | ||||||
|      * @param JacobKiers\OAuth\Client  $client |      * @param JacobKiers\OAuth\Client  $client | ||||||
|      * @param string                   $token_type |      * @param string                   $token_type | ||||||
|      * |      * | ||||||
| @@ -245,7 +245,7 @@ private function getToken(RequestInterface $request, Client $client, $token_type | |||||||
|      * |      * | ||||||
|      * Should determine the signature method appropriately |      * Should determine the signature method appropriately | ||||||
|      * |      * | ||||||
|      * @param JacobKiers\OAuth\RequestInterface $request |      * @param JacobKiers\OAuth\RequestInterfaceInterface $request | ||||||
|      * @param JacobKiers\OAuth\Client  $client |      * @param JacobKiers\OAuth\Client  $client | ||||||
|      * @param JacobKiers\OAuth\Token   $token |      * @param JacobKiers\OAuth\Token   $token | ||||||
|      * |      * | ||||||
|   | |||||||
| @@ -36,13 +36,13 @@ abstract public function getName(); | |||||||
|      * the encoding is handled in OAuthRequest when the final |      * the encoding is handled in OAuthRequest when the final | ||||||
|      * request is serialized. |      * request is serialized. | ||||||
|      * |      * | ||||||
|      * @param JacobKiers\OAuth\Request $request |      * @param JacobKiers\OAuth\RequestInterface $request | ||||||
|      * @param JacobKiers\OAuth\Client  $client |      * @param JacobKiers\OAuth\Client  $client | ||||||
|      * @param JacobKiers\OAuth\Token   $token |      * @param JacobKiers\OAuth\Token   $token | ||||||
|      * |      * | ||||||
|      * @return string |      * @return string | ||||||
|      */ |      */ | ||||||
|     abstract public function buildSignature(Request $request, Client $client, Token $token = null); |     abstract public function buildSignature(RequestInterface $request, Client $client, Token $token = null); | ||||||
|  |  | ||||||
|     /** |     /** | ||||||
|      * Get the signature key, made up of client and optionally token shared secrets. |      * Get the signature key, made up of client and optionally token shared secrets. | ||||||
| @@ -66,14 +66,14 @@ public function getSignatureKey(Client $client, Token $token = null) | |||||||
|     /** |     /** | ||||||
|      * Verifies that a given signature is correct. |      * Verifies that a given signature is correct. | ||||||
|      * |      * | ||||||
|      * @param JacobKiers\OAuth\Request  $request |      * @param JacobKiers\OAuth\RequestInterface  $request | ||||||
|      * @param JacobKiers\OAuth\Consumer $client |      * @param JacobKiers\OAuth\Consumer $client | ||||||
|      * @param JacobKiers\OAuth\Token    $token |      * @param JacobKiers\OAuth\Token    $token | ||||||
|      * @param string                   $signature |      * @param string                   $signature | ||||||
|      * |      * | ||||||
|      * @return bool |      * @return bool | ||||||
|      */ |      */ | ||||||
|     public function checkSignature(Request $request, Client $client, Token $token, $signature) |     public function checkSignature(RequestInterface $request, Client $client, Token $token, $signature) | ||||||
|     { |     { | ||||||
|         $built = $this->buildSignature($request, $client, $token); |         $built = $this->buildSignature($request, $client, $token); | ||||||
|         return $built == $signature; |         return $built == $signature; | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user
	 Jacob Kiers
					Jacob Kiers