From 5d01ea8755874086e85b9a04938ea7209b03cbcc Mon Sep 17 00:00:00 2001
From: Josh <josh.t.richards@gmail.com>
Date: Fri, 6 Mar 2026 08:47:31 -0500
Subject: [PATCH] chore: migrate AuditLogger to IAppConfig

Signed-off-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: Carl Schwan <carlschwan@kde.org>
---
 apps/admin_audit/lib/AppInfo/Application.php   |  6 +-----
 apps/admin_audit/lib/AuditLogger.php           |  9 +++++++--
 apps/admin_audit/lib/BackgroundJobs/Rotate.php |  4 +++-
 build/psalm-baseline.xml                       | 10 ----------
 4 files changed, 11 insertions(+), 18 deletions(-)

diff --git a/apps/admin_audit/lib/AppInfo/Application.php b/apps/admin_audit/lib/AppInfo/Application.php
index 8c936b5accd..bea0817baab 100644
--- a/apps/admin_audit/lib/AppInfo/Application.php
+++ b/apps/admin_audit/lib/AppInfo/Application.php
@@ -53,9 +53,7 @@ use OCP\Group\Events\GroupCreatedEvent;
 use OCP\Group\Events\GroupDeletedEvent;
 use OCP\Group\Events\UserAddedEvent;
 use OCP\Group\Events\UserRemovedEvent;
-use OCP\IConfig;
 use OCP\Log\Audit\CriticalActionPerformedEvent;
-use OCP\Log\ILogFactory;
 use OCP\Preview\BeforePreviewFetchedEvent;
 use OCP\Share;
 use OCP\Share\Events\ShareCreatedEvent;
@@ -81,9 +79,7 @@ class Application extends App implements IBootstrap {
 
 	#[\Override]
 	public function register(IRegistrationContext $context): void {
-		$context->registerService(IAuditLogger::class, function (ContainerInterface $c) {
-			return new AuditLogger($c->get(ILogFactory::class), $c->get(IConfig::class));
-		});
+		$context->registerServiceAlias(IAuditLogger::class, AuditLogger::class);
 
 		$context->registerEventListener(CriticalActionPerformedEvent::class, CriticalActionPerformedEventListener::class);
 
diff --git a/apps/admin_audit/lib/AuditLogger.php b/apps/admin_audit/lib/AuditLogger.php
index 147f228851a..52b228385b9 100644
--- a/apps/admin_audit/lib/AuditLogger.php
+++ b/apps/admin_audit/lib/AuditLogger.php
@@ -9,6 +9,7 @@ declare(strict_types=1);
 
 namespace OCA\AdminAudit;
 
+use OCP\AppFramework\Services\IAppConfig;
 use OCP\IConfig;
 use OCP\Log\ILogFactory;
 use Psr\Log\LoggerInterface;
@@ -20,7 +21,11 @@ class AuditLogger implements IAuditLogger {
 
 	private LoggerInterface $parentLogger;
 
-	public function __construct(ILogFactory $logFactory, IConfig $config) {
+	public function __construct(
+		ILogFactory $logFactory,
+		IAppConfig $appConfig,
+		IConfig $config,
+	) {
 		$auditType = $config->getSystemValueString('log_type_audit', 'file');
 		$defaultTag = $config->getSystemValueString('syslog_tag', 'Nextcloud');
 		$auditTag = $config->getSystemValueString('syslog_tag_audit', $defaultTag);
@@ -29,7 +34,7 @@ class AuditLogger implements IAuditLogger {
 		if ($auditType === 'file' && !$logFile) {
 			$default = $config->getSystemValue('datadirectory', \OC::$SERVERROOT . '/data') . '/audit.log';
 			// Legacy way was appconfig, now it's paralleled with the normal log config
-			$logFile = $config->getAppValue('admin_audit', 'logfile', $default);
+			$logFile = $appConfig->getAppValueString('logfile', $default);
 		}
 
 		$this->parentLogger = $logFactory->getCustomPsrLogger($logFile, $auditType, $auditTag);
diff --git a/apps/admin_audit/lib/BackgroundJobs/Rotate.php b/apps/admin_audit/lib/BackgroundJobs/Rotate.php
index 85133c009ad..2c0a153f93b 100644
--- a/apps/admin_audit/lib/BackgroundJobs/Rotate.php
+++ b/apps/admin_audit/lib/BackgroundJobs/Rotate.php
@@ -8,6 +8,7 @@ declare(strict_types=1);
 
 namespace OCA\AdminAudit\BackgroundJobs;
 
+use OCP\AppFramework\Services\IAppConfig;
 use OCP\AppFramework\Utility\ITimeFactory;
 use OCP\BackgroundJob\TimedJob;
 use OCP\IConfig;
@@ -18,6 +19,7 @@ class Rotate extends TimedJob {
 
 	public function __construct(
 		ITimeFactory $time,
+		private IAppConfig $appConfig,
 		private IConfig $config,
 	) {
 		parent::__construct($time);
@@ -28,7 +30,7 @@ class Rotate extends TimedJob {
 	#[\Override]
 	protected function run($argument): void {
 		$default = $this->config->getSystemValue('datadirectory', \OC::$SERVERROOT . '/data') . '/audit.log';
-		$this->filePath = $this->config->getAppValue('admin_audit', 'logfile', $default);
+		$this->filePath = $this->appConfig->getAppValueString('logfile', $default);
 
 		if ($this->filePath === '') {
 			// default log file, nothing to do
diff --git a/build/psalm-baseline.xml b/build/psalm-baseline.xml
index fe478aab69d..db3939c3928 100644
--- a/build/psalm-baseline.xml
+++ b/build/psalm-baseline.xml
@@ -20,16 +20,6 @@
       <code><![CDATA[Util::connectHook(Share::class, 'share_link_access', $shareActions, 'shareAccessed')]]></code>
     </DeprecatedMethod>
   </file>
-  <file src="apps/admin_audit/lib/AuditLogger.php">
-    <DeprecatedMethod>
-      <code><![CDATA[getAppValue]]></code>
-    </DeprecatedMethod>
-  </file>
-  <file src="apps/admin_audit/lib/BackgroundJobs/Rotate.php">
-    <DeprecatedMethod>
-      <code><![CDATA[getAppValue]]></code>
-    </DeprecatedMethod>
-  </file>
   <file src="apps/cloud_federation_api/lib/Controller/RequestHandlerController.php">
     <DeprecatedMethod>
       <code><![CDATA[Util::emitHook(