Merge pull request #56090 from nextcloud/fix/mailer/log-errors

kesselb · web-flow · commit 4ecd21784be2 · 2025-10-30T12:02:29.000+01:00
fix(Mailer): Log errors
diff --git a/lib/private/Mail/Mailer.php b/lib/private/Mail/Mailer.php
@@ -205,7 +205,7 @@ public function send(IMessage $message): array {
 			$mailer->send($message->getSymfonyEmail());
 		} catch (TransportExceptionInterface $e) {
 			$logMessage = sprintf('Sending mail to "%s" with subject "%s" failed', print_r($message->getTo(), true), $message->getSubject());
-			$this->logger->debug($logMessage, ['app' => 'core', 'exception' => $e]);
+			$this->logger->error($logMessage, ['app' => 'core', 'exception' => $e]);
 			if ($debugMode) {
 				$this->logger->debug($e->getDebug(), ['app' => 'core']);
 			}

Original file line number	Diff line number	Diff line change
`@@ -205,7 +205,7 @@ public function send(IMessage $message): array {`
`205`	`205`	`$mailer->send($message->getSymfonyEmail());`
`206`	`206`	`} catch (TransportExceptionInterface $e) {`
`207`	`207`	`$logMessage = sprintf('Sending mail to "%s" with subject "%s" failed', print_r($message->getTo(), true), $message->getSubject());`
`208`		`- $this->logger->debug($logMessage, ['app' => 'core', 'exception' => $e]);`
	`208`	`+ $this->logger->error($logMessage, ['app' => 'core', 'exception' => $e]);`
`209`	`209`	`if ($debugMode) {`
`210`	`210`	`$this->logger->debug($e->getDebug(), ['app' => 'core']);`
`211`	`211`	`}`