Sender email used for sender and reply in SendDisclaimedMailPreflight

[garbast]

In SendDisclaimedMailPreflight the senderEmail and replyToEmail is set with the return of $senderService->getSenderEmail().

Our mail server is allowing only our own email addresses as sender. So we use the ReceiverMailSenderPropertiesGetSenderEmailEvent event and check wether the sender email is allowed and if not overwrite it with a configured email.

With this now the reply is also our own email address. But it would be nice to still use the users email address.

Am I wrong here? Do you like to receive a PR for this?

[mschwemer]

Hi

thanks for your report. This make totally sense.

I am looking forward to your PR.

[garbast]

I had a fresh view on this issue. The solution I proposed isn't realy helpful as it is setting the receivers email as the reply email. This is then processed in the SendMailService like this:

• receiverEmail -> setTo
• replyEmail -> setReplyTo

If the receiver gets the email and he wants to reply, he would reply to himself. Not helpful. So the intend to set the senderEmail also as replyEmail feels natural with a BUT. That's why I propose a different approach.

The double use of getSenderEmail and getSenderName is the core problem. That's why I implement getReplyEmail and getReplyName with own events. Patch will be pushed in a short while.