From: Robin Appelman <robin@icewind.nl>
Date: Thu, 21 Apr 2022 13:23:09 +0000 (+0200)
Subject: cache fullSetupRequired locally
X-Git-Tag: v25.0.0beta1~536^2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=refs%2Fpull%2F32045%2Fhead;p=nextcloud-server.git

cache fullSetupRequired locally

no need to hit redis every time

Signed-off-by: Robin Appelman <robin@icewind.nl>
---

diff --git a/lib/private/Files/SetupManager.php b/lib/private/Files/SetupManager.php
index e0575ea92a5..f1799798ccb 100644
--- a/lib/private/Files/SetupManager.php
+++ b/lib/private/Files/SetupManager.php
@@ -81,6 +81,7 @@ class SetupManager {
 	private LoggerInterface $logger;
 	private IConfig $config;
 	private bool $listeningForProviders;
+	private array $fullSetupRequired = [];
 
 	public function __construct(
 		IEventLogger $eventLogger,
@@ -268,6 +269,7 @@ class SetupManager {
 		$cacheDuration = $this->config->getSystemValueInt('fs_mount_cache_duration', 5 * 60);
 		if ($cacheDuration > 0) {
 			$this->cache->set($user->getUID(), true, $cacheDuration);
+			$this->fullSetupRequired[$user->getUID()] = false;
 		}
 	}
 
@@ -434,7 +436,10 @@ class SetupManager {
 		// we perform a "cached" setup only after having done the full setup recently
 		// this is also used to trigger a full setup after handling events that are likely
 		// to change the available mounts
-		return !$this->cache->get($user->getUID());
+		if (!isset($this->fullSetupRequired[$user->getUID()])) {
+			$this->fullSetupRequired[$user->getUID()] = !$this->cache->get($user->getUID());
+		}
+		return $this->fullSetupRequired[$user->getUID()];
 	}
 
 	/**
@@ -489,6 +494,7 @@ class SetupManager {
 		$this->setupUsers = [];
 		$this->setupUsersComplete = [];
 		$this->setupUserMountProviders = [];
+		$this->fullSetupRequired = [];
 		$this->rootSetup = false;
 		$this->mountManager->clear();
 		$this->eventDispatcher->dispatchTyped(new FilesystemTornDownEvent());