Merge pull request #29685 from nextcloud/backport/29682/stable22

[stable22] Fix missing token update
nextcloud · Nov 12, 2021 · b6adf29 · b6adf29
2 parents 92db6f3 + 6ed9bcc
commit b6adf29
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/lib/private/User/Session.php b/lib/private/User/Session.php
@@ -750,6 +750,7 @@ private function checkTokenCredentials(IToken $dbToken, $token) {
 			}
 
 			$dbToken->setLastCheck($now);
+			$this->tokenProvider->updateToken($dbToken);
 			return true;
 		}
 
@@ -767,6 +768,7 @@ private function checkTokenCredentials(IToken $dbToken, $token) {
 		}
 
 		$dbToken->setLastCheck($now);
+		$this->tokenProvider->updateToken($dbToken);
 		return true;
 	}
 

diff --git a/tests/lib/User/SessionTest.php b/tests/lib/User/SessionTest.php
@@ -1264,7 +1264,7 @@ public function testUpdateAuthTokenLastCheck() {
 		$mapper->expects($this->any())
 			->method('getToken')
 			->willReturn($token);
-		$mapper->expects($this->once())
+		$mapper->expects($this->exactly(2))
 			->method('update');
 		$request
 			->expects($this->any())
@@ -1314,7 +1314,7 @@ public function testNoUpdateAuthTokenLastCheckRecent() {
 		$mapper->expects($this->any())
 			->method('getToken')
 			->willReturn($token);
-		$mapper->expects($this->never())
+		$mapper->expects($this->once())
 			->method('update');
 		$request
 			->expects($this->any())