fix: Dont hijack mailto links

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>

Author: kesselb

lib/Service/Html.php

@@ -126,7 +126,7 @@ public function sanitizeHtmlMailBody(string $mailBody, array $messageParameters,
 		$html = $config->getDefinition('HTML');
 		$html->info_attr_transform_post['imagesrc'] = new TransformImageSrc($this->urlGenerator);
 		$html->info_attr_transform_post['cssbackground'] = new TransformStyleURLs($this->urlGenerator);
-		$html->info_attr_transform_post['htmllinks'] = new TransformHTMLLinks($this->urlGenerator);
+		$html->info_attr_transform_post['htmllinks'] = new TransformHTMLLinks();
 
 		/** @var HTMLPurifier_URIDefinition $uri */
 		$uri = $config->getDefinition('URI');

lib/Service/HtmlPurify/TransformHTMLLinks.php

@@ -13,19 +13,11 @@
 use HTMLPurifier_AttrTransform;
 use HTMLPurifier_Config;
 use HTMLPurifier_Context;
-use OCP\IURLGenerator;
 
 /**
  * Adds target="_blank" to all outbound links.
  */
 class TransformHTMLLinks extends HTMLPurifier_AttrTransform {
-	/** @var IURLGenerator */
-	private $urlGenerator;
-
-	public function __construct(IURLGenerator $urlGenerator) {
-		$this->urlGenerator = $urlGenerator;
-	}
-
 	/**
 	 * @param array $attr
 	 * @param HTMLPurifier_Config $config
@@ -41,11 +33,6 @@ public function transform($attr, $config, $context) {
 		$attr['target'] = '_blank';
 		$attr['rel'] = 'external noopener noreferrer';
 
-		// Open mailto: links in Mail
-		if (stripos($attr['href'], 'mailto:') === 0) {
-			$attr['href'] = $this->urlGenerator->linkToRoute('mail.page.mailto') . '?to=' . substr($attr['href'], 7);
-		}
-
 		return $attr;
 	}
 }