diff --git a/apps/dav/lib/Connector/Sabre/BearerAuth.php b/apps/dav/lib/Connector/Sabre/BearerAuth.php index 5a69d07b0026d..245563cb244e5 100644 --- a/apps/dav/lib/Connector/Sabre/BearerAuth.php +++ b/apps/dav/lib/Connector/Sabre/BearerAuth.php @@ -21,6 +21,7 @@ * along with this program. If not, see . * */ + namespace OCA\DAV\Connector\Sabre; use OCP\IRequest; @@ -62,10 +63,7 @@ private function setupUserFs($userId) { public function validateBearerToken($bearerToken) { \OC_Util::setupFS(); - if (!$this->userSession->isLoggedIn()) { - $this->userSession->tryTokenLogin($this->request); - } - if ($this->userSession->isLoggedIn()) { + if ($this->userSession->tryTokenLogin($this->request)) { return $this->setupUserFs($this->userSession->getUser()->getUID()); } diff --git a/apps/dav/tests/unit/Connector/Sabre/BearerAuthTest.php b/apps/dav/tests/unit/Connector/Sabre/BearerAuthTest.php index 007b5c90295db..a6253fd3e3552 100644 --- a/apps/dav/tests/unit/Connector/Sabre/BearerAuthTest.php +++ b/apps/dav/tests/unit/Connector/Sabre/BearerAuthTest.php @@ -1,4 +1,5 @@ * @@ -22,6 +23,7 @@ * along with this program. If not, see . * */ + namespace OCA\DAV\Tests\unit\Connector\Sabre; use OCA\DAV\Connector\Sabre\BearerAuth; @@ -66,12 +68,8 @@ public function testValidateBearerTokenNotLoggedIn() { public function testValidateBearerToken() { $this->userSession - ->expects($this->at(0)) - ->method('isLoggedIn') - ->willReturn(false); - $this->userSession - ->expects($this->at(2)) - ->method('isLoggedIn') + ->expects($this->once()) + ->method('tryTokenLogin') ->willReturn(true); $user = $this->createMock(IUser::class); $user