Merge pull request #10433 from nextcloud/fix/mailto/bcc-recipients

fix(mailto): Handle BCC recipients only

lib/Controller/PageController.php

@@ -411,7 +411,10 @@ public function filteredDraft(string $filter, int $mailboxId, int $draftId): Tem
 	 */
 	public function compose(string $uri): RedirectResponse {
 		$parts = parse_url($uri);
-		$params = ['to' => $parts['path']];
+		$params = [];
+		if (isset($parts['path'])) {
+			$params['to'] = $parts['path'];
+		}
 		if (isset($parts['query'])) {
 			$parts = explode('&', $parts['query']);
 			foreach ($parts as $part) {

src/components/MailboxThread.vue

@@ -362,6 +362,7 @@ export default {
 						accountId,
 						to: this.stringToRecipients(this.$route.query.to),
 						cc: this.stringToRecipients(this.$route.query.cc),
+						bcc: this.stringToRecipients(this.$route.query.bcc),
 						subject: this.$route.query.subject || '',
 						body: this.$route.query.body ? detect(this.$route.query.body) : html(''),
 					},

tests/Unit/Controller/PageControllerTest.php

@@ -40,6 +40,7 @@
 use PHPUnit\Framework\MockObject\MockObject;
 use Psr\Container\ContainerInterface;
 use Psr\Log\LoggerInterface;
+use function urlencode;
 
 class PageControllerTest extends TestCase {
 	/** @var string */
@@ -380,6 +381,17 @@ public function testComposeWithCc() {
 		$this->assertEquals($expected, $response);
 	}
 
+	public function testComposeBcc() {
+		$bcc = '[email protected]';
+		$uri = "mailto:?bcc=$bcc";
+
+		$expected = new RedirectResponse('?bcc=' . urlencode($bcc));
+
+		$response = $this->controller->compose($uri);
+
+		$this->assertEquals($expected, $response);
+	}
+
 	public function testComposeWithBcc() {
 		$address = '[email protected]';
 		$bcc = '[email protected]';