only update last login timestamp with minute percision

Signed-off-by: Robin Appelman <[email protected]>
LinneyS · Jul 20, 2022 · 9e34a21 · 9e34a21
1 parent de08b53
commit 9e34a21
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/lib/private/User/User.php b/lib/private/User/User.php
@@ -244,10 +244,15 @@ public function getLastLogin() {
 	 * updates the timestamp of the most recent login of this user
 	 */
 	public function updateLastLoginTimestamp() {
-		$firstTimeLogin = ($this->getLastLogin() === 0);
-		$this->lastLogin = time();
-		$this->config->setUserValue(
-			$this->uid, 'login', 'lastLogin', (string)$this->lastLogin);
+		$previousLogin = $this->getLastLogin();
+		$now = time();
+		$firstTimeLogin = $previousLogin === 0;
+
+		if ($now - $previousLogin > 60) {
+			$this->lastLogin = time();
+			$this->config->setUserValue(
+				$this->uid, 'login', 'lastLogin', (string)$this->lastLogin);
+		}
 
 		return $firstTimeLogin;
 	}