$header was always true due to previous if statement

diff --git a/lib/apiauthaction.php b/lib/apiauthaction.php
index 683d42e7055d0d53fa2ab1b80335f938736c005c..a7b212d79ad55a1913ab12a7828b195589ceb1b2 100644 (file)
--- a/lib/apiauthaction.php
+++ b/lib/apiauthaction.php
@@ -310,7 +310,7 @@ class ApiAuthAction extends ApiAction
             // By default, basic auth users have rw access
             $this->access = self::READ_WRITE;
 
-            if (!$this->auth_user instanceof User && ($required || isset($_SERVER['PHP_AUTH_USER']))) {
+            if (!$this->auth_user instanceof User && isset($_SERVER['PHP_AUTH_USER'])) {
                 $msg = sprintf(
                     "basic auth nickname = %s",
                     $this->auth_user_nickname