Better exception management for token retrieval.

Author: VincentChalnot

Client/OAuthRequestFactory.php

@@ -55,6 +55,9 @@ static function (array $data) {
     protected function updateToken(RequestInterface $request, callable $method): OAuthTokenInterface
     {
         $response = $this->client->sendRequest($request);
+        if (200 !== $response->getStatusCode()) {
+            throw OAuthAuthenticationException::createFromTokenResponse($response, $this->tokenRequestPath);
+        }
 
         $content = (string) $response->getBody();
         if (!$content) {

Exception/OAuthAuthenticationException.php

@@ -12,13 +12,13 @@
 /**
  * Thrown when unable to authenticate against the OAuth server API
  */
-class OAuthAuthenticationException extends \RuntimeException
+class OAuthAuthenticationException extends RequestFailedException
 {
     public static function createFromTokenResponse(ResponseInterface $response, string $tokenRequestPath): self
     {
         $m = "Unable to get OAuth token from remote server '{$tokenRequestPath}': ";
-        $m .= "{$response->getStatusCode()} {$response->getReasonPhrase()}\n{$response->getBody()}";
+        $m .= " {$response->getReasonPhrase()}\n";
 
-        return new self($m);
+        return new self($response, $m);
     }
 }