Use 'From' email address for the sendmail POST user ID

Attached patch implements the following changes:

1. Use 'From' email address for the sendmail POST user id to allow a dynamic sender.
2. Removes 'from' from the mail body, as this information is added in the mail process and derived from the account information.

Reroll after 📌 Add cc, bcc, and reply-to to message Fixed was committed.

apaderno → closed merge request !3

Thank you for the patch!

This change introduced a breaking change on my side as the regex forces the "From" address to be surrounded with < and >. On our end, all the From address are just simple email addresses without the <>.

For me this is also a breaking change. I also think you should never allow to use the from address because it can cause e-mails not being send, or seen as SPAM. To avoid this, the reply-to header/property is used. In this way you can send from a general mailbox, but the receiver can still reply to the mail.

The webform module also makes use of the reply-to header and populates it with the from address if not filled in.

I suggest a revert for this

Yes, the email used to send emails must contain the domain name used for the site. Otherwise, the email would be flagged as spam.

I could imagine the module to use a setting for the email to use for sending emails, but I also understand that probably most sites would use the same email set for the site email.

I guess this has been reverted with 🐛 Email not sent when address in format "Name " Fixed .