fix(setup): ignore long session login during installation

Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
nextcloud · Aug 27, 2024 · 135b270 · 135b270
1 parent 6c73b11
commit 135b270
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 4 deletions.
diff --git a/lib/base.php b/lib/base.php
@@ -448,10 +448,15 @@ public static function initSession(): void {
 		$sessionName = OC_Util::getInstanceId();
 
 		try {
+			$logger = null;
+			if (Server::get(\OC\SystemConfig::class)->getValue('installed', false)) {
+				$logger = logger('core');
+			}
+
 			// set the session name to the instance id - which is unique
 			$session = new \OC\Session\Internal(
 				$sessionName,
-				logger('core'),
+				$logger,
 			);
 
 			$cryptoWrapper = Server::get(\OC\Session\CryptoWrapper::class);

diff --git a/lib/private/Session/Internal.php b/lib/private/Session/Internal.php
@@ -53,8 +53,10 @@ class Internal extends Session {
 	 * @param string $name
 	 * @throws \Exception
 	 */
-	public function __construct(string $name,
-		private LoggerInterface $logger) {
+	public function __construct(
+		string $name,
+		private ?LoggerInterface $logger,
+	) {
 		set_error_handler([$this, 'trapError']);
 		$this->invoke('session_name', [$name]);
 		$this->invoke('session_cache_limiter', ['']);
@@ -228,7 +230,7 @@ private function invoke(string $functionName, array $parameters = [], bool $sile
 					$timeSpent > 0.5 => ILogger::INFO,
 					default => ILogger::DEBUG,
 				};
-				$this->logger->log(
+				$this->logger?->log(
 					$logLevel,
 					"Slow session operation $functionName detected",
 					[