Using a Name <Email> in Reply-to results in a RfcComplianceException

I added a merge request with a simple fix, but this may not be the most appropriate resolution, and it does not have any tests.

Thanks for the MR. I think this approach sounds reasonable. There's already an indexed array below that code for addresses without name.

My only issue is that I think we should use $mailbox_raw instead of reconstructing the address. In my code, I'm removing the quotes from the name. I'm not totally sure where I got that from. Either Swiftmailer or Symfony Mailer module code probably. I'm not sure it's appropriate. We could probably just get rid of the line creating $mailbox_name and use $mailbox_raw once we validate the address.

My only issue is that I think we should use $mailbox_raw instead of reconstructing the address. In my code, I'm removing the quotes from the name. I'm not totally sure where I got that from. Either Swiftmailer or Symfony Mailer module code probably. I'm not sure it's appropriate. We could probably just get rid of the line creating $mailbox_name and use $mailbox_raw once we validate the address.

+1. I was thinking this as well, but I wasn't sure about using a variable with the name "raw" in it. :-)

I committed the updated version of the code.

Automatically closed - issue fixed for 2 weeks with no activity.