Add a status code to the IdentityProviderException

Previously this exception would always have `0` as the status code, making it impossible to know what was the status of the response that caused that exception.
stevenmaguire · Oct 24, 2023 · 1b690b7 · 1b690b7
1 parent fdd159f
commit 1b690b7
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/Provider/Keycloak.php b/src/Provider/Keycloak.php
@@ -243,7 +243,7 @@ protected function checkResponse(ResponseInterface $response, $data)
             if (isset($data['error_description'])) {
                 $error .= ': '.$data['error_description'];
             }
-            throw new IdentityProviderException($error, 0, $data);
+            throw new IdentityProviderException($error, $response->getStatusCode(), $data);
         }
     }
 

diff --git a/test/src/Provider/KeycloakTest.php b/test/src/Provider/KeycloakTest.php
@@ -512,7 +512,9 @@ public function testErrorResponse()
                 ->andReturn($accessTokenResponseStream);
             $response
                 ->shouldReceive('getHeader')
-                ->andReturn(['content-type' => 'json']);
+            $response
+                ->shouldReceive('getStatusCode')
+                ->andReturn(401);
 
             $client = m::mock('GuzzleHttp\ClientInterface');
             $client