Fixed bug in basic auth.

Signed-off-by: Jason Lewis <[email protected]>

Author: jasonlewis

src/Auth/BasicProvider.php

@@ -47,7 +47,7 @@ public function authenticate(Request $request, Route $route)
     {
         $this->validateAuthorizationHeader($request);
 
-        if (is_null($response = $this->auth->onceBasic($this->identifier, $request)) || $response->getStatusCode() === 401) {
+        if (($response = $this->auth->onceBasic($this->identifier, $request)) && $response->getStatusCode() === 401) {
             throw new UnauthorizedHttpException('Basic', 'Invalid authentication credentials.');
         }
 

tests/Auth/BasicProviderTest.php

@@ -37,24 +37,11 @@ public function testInvalidBasicCredentialsThrowsException()
     }
 
 
-    /**
-     * @expectedException \Symfony\Component\HttpKernel\Exception\UnauthorizedHttpException
-     */
-    public function testNullResponseFromAuthManagerThrowsException()
-    {
-        $request = Request::create('GET', '/', [], [], [], ['HTTP_AUTHORIZATION' => 'Basic 12345']);
-
-        $this->auth->shouldReceive('onceBasic')->once()->with('email', $request)->andReturn(null);
-
-        $this->provider->authenticate($request, new Route(['GET'], '/', []));
-    }
-
-
     public function testValidCredentialsReturnsUser()
     {
         $request = Request::create('GET', '/', [], [], [], ['HTTP_AUTHORIZATION' => 'Basic 12345']);
 
-        $this->auth->shouldReceive('onceBasic')->once()->with('email', $request)->andReturn(new Response('', 200));
+        $this->auth->shouldReceive('onceBasic')->once()->with('email', $request)->andReturn(null);
         $this->auth->shouldReceive('user')->once()->andReturn('foo');
 
         $this->assertEquals('foo', $this->provider->authenticate($request, new Route(['GET'], '/', [])));