Added check for token NULL value before saving following 401

If the token header is NULL (as it would be for a 401) and this is saved, the subsequent authorisation attempt is impossible since the existing token has been overwritten.
damonjones · Sep 11, 2013 · b8f4e18 · b8f4e18
1 parent f01752b
commit b8f4e18
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/lib/YDD/Vebra/API.php b/lib/YDD/Vebra/API.php
@@ -153,9 +153,11 @@ public function execute($url, $secondAttempt = false)
                 $tokenResponse = $this->messageFactory->createResponse();
                 $this->client->send($tokenRequest, $tokenResponse);
 
-                // save the token
+                // if we have a token, save it
                 $token = $tokenResponse->getHeader('Token');
-                $this->tokenStorage->setToken(trim($token));
+                if ( ! is_null($token)) {
+                    $this->tokenStorage->setToken(trim($token));
+                }
 
                 // try again
                 return $this->execute($url, true);