From 73615a77d450fd9ac484046b9b66528b89d453a0 Mon Sep 17 00:00:00 2001 From: Vitor Mattos <1079143+vitormattos@users.noreply.github.com> Date: Wed, 17 Dec 2025 02:36:56 -0300 Subject: [PATCH] fix: use correct Nextcloud instanceid in PKI path initialization The initializePkiConfigPath method was using a variable named 'instanceId' from getSystemValue('instanceid') which could be confused with LibreSign's instance_id. Changed to 'systemInstanceId' for clarity and consistency with getConfigPathByParams method. This ensures the PKI directories are created in the correct appdata path using Nextcloud's system instance ID, while the directory name itself contains LibreSign's CA identifier with its own instance ID. Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com> --- lib/Handler/CertificateEngine/AEngineHandler.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/Handler/CertificateEngine/AEngineHandler.php b/lib/Handler/CertificateEngine/AEngineHandler.php index 99faa7ed3..5854a11d0 100644 --- a/lib/Handler/CertificateEngine/AEngineHandler.php +++ b/lib/Handler/CertificateEngine/AEngineHandler.php @@ -363,8 +363,8 @@ abstract class AEngineHandler implements IEngineHandler { } $pkiDirName = $this->caIdentifierService->generatePkiDirectoryName($caId); $dataDir = $this->config->getSystemValue('datadirectory', \OC::$SERVERROOT . '/data/'); - $instanceId = $this->config->getSystemValue('instanceid'); - $pkiPath = $dataDir . '/appdata_' . $instanceId . '/libresign/' . $pkiDirName; + $systemInstanceId = $this->config->getSystemValue('instanceid'); + $pkiPath = $dataDir . '/appdata_' . $systemInstanceId . '/libresign/' . $pkiDirName; if (!is_dir($pkiPath)) { $this->createDirectoryWithCorrectOwnership($pkiPath);