fixup! fixup! feat: per account imap and smtp debugging

Signed-off-by: SebastianKrupinski <[email protected]>
nextcloud · Nov 7, 2024 · 2c0f6fe · 2c0f6fe
1 parent 6f37da9
commit 2c0f6fe
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/lib/IMAP/IMAPClientFactory.php b/lib/IMAP/IMAPClientFactory.php
@@ -118,7 +118,7 @@ public function getClient(Account $account, bool $useCache = true): Horde_Imap_C
 			];
 		}
 		$debug = array_merge(
-			explode('|', $this->config->getSystemValue('MAIL_DEBUG')),
+			explode('|', $this->config->getSystemValueString('MAIL_DEBUG')),
 			explode('|', $account->getDebug())
 		);
 		if (in_array('imap', $debug) || in_array('imap-full', $debug)) {

diff --git a/lib/SMTP/SmtpClientFactory.php b/lib/SMTP/SmtpClientFactory.php
@@ -78,7 +78,7 @@ public function create(Account $account): Horde_Mail_Transport {
 			);
 		}
 		$debug = array_merge(
-			explode('|', $this->config->getSystemValue('MAIL_DEBUG', '')),
+			explode('|', $this->config->getSystemValueString('MAIL_DEBUG')),
 			explode('|', $account->getDebug())
 		);
 		if (in_array('smtp', $debug)) {

diff --git a/tests/Unit/SMTP/SmtpClientFactoryTest.php b/tests/Unit/SMTP/SmtpClientFactoryTest.php
@@ -69,13 +69,16 @@ public function testSmtpTransport() {
 			->method('getSystemValue')
 			->willReturnMap([
 				['app.mail.transport', 'smtp', 'smtp'],
-				['debug', false, false],
 				['app.mail.smtp.timeout', 20, 2],
 			]);
 		$this->config->expects($this->any())
 			->method('getSystemValueBool')
 			->with('app.mail.verify-tls-peer', true)
 			->willReturn(true);
+		$this->config->expects($this->any())
+			->method('getSystemValueString')
+			->with('MAIL_DEBUG')
+			->willReturn('');
 		$this->crypto->expects($this->once())
 			->method('decrypt')
 			->with('obenc')