feat: jmap support - 2b

Signed-off-by: SebastianKrupinski <krupinskis05@gmail.com>

Author: SebastianKrupinski

lib/AppInfo/Application.php

@@ -17,7 +17,6 @@
 use OCA\Mail\Contracts\IDkimService;
 use OCA\Mail\Contracts\IDkimValidator;
 use OCA\Mail\Contracts\IMailSearch;
-use OCA\Mail\Contracts\IMailTransmission;
 use OCA\Mail\Contracts\ITrustedSenderService;
 use OCA\Mail\Contracts\IUserPreferences;
 use OCA\Mail\Dashboard\ImportantMailWidget;
@@ -61,7 +60,6 @@
 use OCA\Mail\Service\AvatarService;
 use OCA\Mail\Service\DkimService;
 use OCA\Mail\Service\DkimValidator;
-use OCA\Mail\Service\MailTransmission;
 use OCA\Mail\Service\Search\MailSearch;
 use OCA\Mail\Service\TrustedSenderService;
 use OCA\Mail\Service\UserPreferenceService;
@@ -119,7 +117,6 @@ public function register(IRegistrationContext $context): void {
 		$context->registerServiceAlias(IAvatarService::class, AvatarService::class);
 		$context->registerServiceAlias(IAttachmentService::class, AttachmentService::class);
 		$context->registerServiceAlias(IMailSearch::class, MailSearch::class);
-		$context->registerServiceAlias(IMailTransmission::class, MailTransmission::class);
 		$context->registerServiceAlias(ITrustedSenderService::class, TrustedSenderService::class);
 		$context->registerServiceAlias(IUserPreferences::class, UserPreferenceService::class);
 		$context->registerServiceAlias(IDkimService::class, DkimService::class);

lib/Contracts/IMailTransmission.php

@@ -12,24 +12,25 @@
 use OCA\Mail\Account;
 use OCA\Mail\Db\LocalMessage;
 use OCA\Mail\Db\Mailbox;
+use OCA\Mail\Db\Message;
 
 interface ITransmissionConnector {
 
 	/**
-	 * Send a composed message.
+	 * Send a composed message and file a copy in the sent mailbox.
 	 */
-	public function sendMessage(Account $account, LocalMessage $message): void;
+	public function sendMessage(Account $account, LocalMessage $message, Mailbox $sentMailbox): void;
 
 	/**
-	 * Save a message as a draft.
+	 * Send a Message Disposition Notification (read receipt).
 	 */
-	public function saveDraft(Account $account, LocalMessage $message): void;
+	public function sendMdn(Account $account, Mailbox $mailbox, Message $message): void;
 
 	/**
-	 * Store a raw RFC 5322 message into a mailbox.
+	 * Store a message in a mailbox.
 	 *
-	 * @param string[] $flags Flags to set on the stored message.
-	 * @return int|null The UID of the stored message, if known.
+	 * @param string[] $flags JMAP-style keywords to set on the stored message (e.g. ['$draft']).
+	 *                        Each connector maps these to its native flag format internally.
 	 */
-	public function saveRawMessageToMailbox(Account $account, Mailbox $mailbox, string $raw, array $flags = []): ?int;
+	public function saveMessage(Account $account, Mailbox $mailbox, LocalMessage $message, array $flags = []): void;
 }

lib/Contracts/ITransmissionConnector.php

@@ -10,23 +10,19 @@
 
 namespace OCA\Mail\Controller;
 
-use Horde_Imap_Client;
 use OCA\Mail\Account;
 use OCA\Mail\AppInfo\Application;
-use OCA\Mail\Contracts\IMailTransmission;
 use OCA\Mail\Db\Mailbox;
 use OCA\Mail\Exception\ClientException;
 use OCA\Mail\Exception\CouldNotConnectException;
 use OCA\Mail\Exception\ServiceException;
 use OCA\Mail\Http\JsonResponse as MailJsonResponse;
 use OCA\Mail\Http\TrapError;
 use OCA\Mail\IMAP\MailboxSync;
-use OCA\Mail\Model\NewMessageData;
 use OCA\Mail\Service\AccountService;
 use OCA\Mail\Service\AliasesService;
 use OCA\Mail\Service\MailManager;
 use OCA\Mail\Service\SetupService;
-use OCA\Mail\Service\Sync\SyncService;
 use OCP\AppFramework\Controller;
 use OCP\AppFramework\Http;
 use OCP\AppFramework\Http\Attribute\OpenAPI;
@@ -45,10 +41,8 @@ class AccountsController extends Controller {
 	private LoggerInterface $logger;
 	private IL10N $l10n;
 	private AliasesService $aliasesService;
-	private IMailTransmission $mailTransmission;
 	private SetupService $setup;
 	private MailManager $mailManager;
-	private SyncService $syncService;
 	private IConfig $config;
 	private IRemoteHostValidator $hostValidator;
 	private MailboxSync $mailboxSync;
@@ -61,10 +55,8 @@ public function __construct(
 		LoggerInterface $logger,
 		IL10N $l10n,
 		AliasesService $aliasesService,
-		IMailTransmission $mailTransmission,
 		SetupService $setup,
 		MailManager $mailManager,
-		SyncService $syncService,
 		IConfig $config,
 		IRemoteHostValidator $hostValidator,
 		MailboxSync $mailboxSync,
@@ -76,10 +68,8 @@ public function __construct(
 		$this->logger = $logger;
 		$this->l10n = $l10n;
 		$this->aliasesService = $aliasesService;
-		$this->mailTransmission = $mailTransmission;
 		$this->setup = $setup;
 		$this->mailManager = $mailManager;
-		$this->syncService = $syncService;
 		$this->config = $config;
 		$this->hostValidator = $hostValidator;
 		$this->mailboxSync = $mailboxSync;
@@ -398,59 +388,6 @@ public function create(string $accountName,
 		);
 	}
 
-	/**
-	 * @NoAdminRequired
-	 *
-	 * @return JSONResponse
-	 *
-	 * @throws ClientException
-	 */
-	#[TrapError]
-	public function draft(int $id,
-		string $subject,
-		string $body,
-		string $to,
-		string $cc,
-		string $bcc,
-		bool $isHtml = true,
-		?int $draftId = null): JSONResponse {
-		if ($draftId === null) {
-			$this->logger->info("Saving a new draft in account <$id>");
-		} else {
-			$this->logger->info("Updating draft <$draftId> in account <$id>");
-		}
-
-		$account = $this->accountService->find($this->currentUserId, $id);
-		$previousDraft = null;
-		if ($draftId !== null) {
-			try {
-				$previousDraft = $this->mailManager->getMessage($this->currentUserId, $draftId);
-			} catch (ClientException $e) {
-				$this->logger->info("Draft {$draftId} could not be loaded: {$e->getMessage()}");
-			}
-		}
-		$messageData = NewMessageData::fromRequest($account, $subject, $body, $to, $cc, $bcc, [], $isHtml);
-
-		try {
-			/** @var Mailbox $draftsMailbox */
-			[, $draftsMailbox, $newUID] = $this->mailTransmission->saveDraft($messageData, $previousDraft);
-			$this->syncService->syncMailbox(
-				$account,
-				$draftsMailbox,
-				Horde_Imap_Client::SYNC_NEWMSGSUIDS,
-				false,
-				null,
-				[]
-			);
-			return new JSONResponse([
-				'id' => $this->mailManager->getMessageIdForUid($draftsMailbox, $newUID)
-			]);
-		} catch (ClientException|ServiceException $ex) {
-			$this->logger->error('Saving draft failed: ' . $ex->getMessage());
-			throw $ex;
-		}
-	}
-
 	/**
 	 * @NoAdminRequired
 	 *

lib/Controller/AccountsController.php

@@ -15,8 +15,8 @@
 use OCA\Mail\Attachment;
 use OCA\Mail\Contracts\IDkimService;
 use OCA\Mail\Contracts\IMailSearch;
-use OCA\Mail\Contracts\IMailTransmission;
 use OCA\Mail\Contracts\ITrustedSenderService;
+use OCA\Mail\Protocol\ProtocolFactory;
 use OCA\Mail\Contracts\IUserPreferences;
 use OCA\Mail\Db\Message;
 use OCA\Mail\Exception\ClientException;
@@ -67,7 +67,7 @@ class MessagesController extends Controller {
 	private IURLGenerator $urlGenerator;
 	private ContentSecurityPolicyNonceManager $nonceManager;
 	private ITrustedSenderService $trustedSenderService;
-	private IMailTransmission $mailTransmission;
+	private ProtocolFactory $protocolFactory;
 	private SmimeService $smimeService;
 	private IDkimService $dkimService;
 	private IUserPreferences $preferences;
@@ -89,7 +89,7 @@ public function __construct(
 		IURLGenerator $urlGenerator,
 		ContentSecurityPolicyNonceManager $nonceManager,
 		ITrustedSenderService $trustedSenderService,
-		IMailTransmission $mailTransmission,
+		ProtocolFactory $protocolFactory,
 		SmimeService $smimeService,
 		IDkimService $dkimService,
 		IUserPreferences $preferences,
@@ -110,7 +110,7 @@ public function __construct(
 		$this->urlGenerator = $urlGenerator;
 		$this->nonceManager = $nonceManager;
 		$this->trustedSenderService = $trustedSenderService;
-		$this->mailTransmission = $mailTransmission;
+		$this->protocolFactory = $protocolFactory;
 		$this->smimeService = $smimeService;
 		$this->dkimService = $dkimService;
 		$this->preferences = $preferences;
@@ -497,7 +497,7 @@ public function mdn(int $id): JSONResponse {
 		}
 
 		try {
-			$this->mailTransmission->sendMdn($account, $mailbox, $message);
+			$this->protocolFactory->transmissionConnector($account)->sendMdn($account, $mailbox, $message);
 			$this->mailManager->flagMessages($account, $mailbox, '$mdnsent', true, $message);
 		} catch (ServiceException $ex) {
 			$this->logger->error('Sending mdn failed: ' . $ex->getMessage());