fix!: remove mail transport php-mail

Signed-off-by: Daniel Kesselberg <[email protected]>
nextcloud · Dec 17, 2024 · 8d1dd79 · 8d1dd79
1 parent e79c328
commit 8d1dd79
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 29 deletions.
diff --git a/doc/admin.md b/doc/admin.md
@@ -56,6 +56,10 @@ Configure how often Mail keeps users' mailboxes updated in the background in sec
 ```
 
 ### Use php-mail for sending mail
+
+> [!WARNING]
+> Support for using php-mail will be removed with Mail 4.2
+
 You can use the php-mail function to send mails. This is needed for some webhosters (1&1 (1und1)):
 ```php
 'app.mail.transport' => 'php-mail'

diff --git a/lib/SMTP/SmtpClientFactory.php b/lib/SMTP/SmtpClientFactory.php
@@ -43,10 +43,6 @@ public function __construct(IConfig $config,
 	 */
 	public function create(Account $account): Horde_Mail_Transport {
 		$mailAccount = $account->getMailAccount();
-		$transport = $this->config->getSystemValue('app.mail.transport', 'smtp');
-		if ($transport === 'php-mail') {
-			return new Horde_Mail_Transport_Mail();
-		}
 
 		$decryptedPassword = null;
 		if ($mailAccount->getOutboundPassword() !== null) {

diff --git a/lib/SystemConfig.php b/lib/SystemConfig.php