handle RecipientsNotFoundError for calendar event cancellations, #2975

src/calendar/CalendarUpdateDistributor.js

@@ -16,6 +16,7 @@ import type {CalendarEventAttendee} from "../api/entities/tutanota/CalendarEvent
 import {createCalendarEventAttendee} from "../api/entities/tutanota/CalendarEventAttendee"
 import {isTutanotaMailAddress} from "../api/common/RecipientInfo"
 import {createMailAddress} from "../api/entities/tutanota/MailAddress"
+import {RecipientsNotFoundError} from "../api/common/error/RecipientsNotFoundError"
 
 export interface CalendarUpdateDistributor {
 	sendInvite(existingEvent: CalendarEvent, sendMailModel: SendMailModel): Promise<void>;
@@ -72,6 +73,21 @@ export class CalendarMailDistributor implements CalendarUpdateDistributor {
 			body: makeInviteEmailBody(sender, event, message),
 			event,
 			sender
+		}).catch(RecipientsNotFoundError, e => {
+			// we want to delete the event even if the recipient is not an existing tutanota address
+			// and just exclude them from sending out updates but leave the event untouched for other recipients
+			const invalidRecipients = e.message.split("\n")
+			let hasRemovedRecipient = false
+			invalidRecipients.forEach(invalidRecipient => {
+				const recipientInfo = sendMailModel.bccRecipients().find(r => r.mailAddress === invalidRecipient)
+				if (recipientInfo) {
+					hasRemovedRecipient = sendMailModel.removeRecipient(recipientInfo, "bcc", false) || hasRemovedRecipient
+				}
+			})
+			// only try sending again if we successfully removed a recipient and there are still other recipients
+			if (hasRemovedRecipient && sendMailModel.allRecipients().length) {
+				return this.sendCancellation(event, sendMailModel)
+			}
 		})
 	}
 

src/mail/editor/SendMailModel.js

@@ -4,6 +4,7 @@ import {
 	ApprovalStatus,
 	ConversationType,
 	MailFolderType,
+	MailMethod,
 	MAX_ATTACHMENT_SIZE,
 	OperationType,
 	ReplyType
@@ -751,8 +752,12 @@ export class SendMailModel {
 			// catch all of the badness
 			.catch(RecipientNotResolvedError, () => {throw new UserError("tooManyAttempts_msg")})
 			.catch(RecipientsNotFoundError, (e) => {
-				let invalidRecipients = e.message
-				throw new UserError(() => lang.get("invalidRecipients_msg") + "\n" + invalidRecipients)
+				if (mailMethod === MailMethod.ICAL_CANCEL) {
+					//in case of calendar event cancellation we willremove invalid recipients and then delete the event without sending updates
+					throw e
+				} else {
+					throw new UserError(() => lang.get("invalidRecipients_msg") + "\n" + e.message)
+				}
 			})
 			.catch(TooManyRequestsError, () => {throw new UserError(tooManyRequestsError)})
 			.catch(AccessBlockedError, e => {