[Mailer] BCC doesn't work with AWS SES

[Pixelshaped]

Symfony version(s) affected: 4.4.7

Description
I was able to send emails to the to() address, but bcc() seems ignored.
Using templated emails.
The URL I use for the DSN is one of type:
MAILER_DSN=ses://ACCESS_KEY:SECRET_KEY@default?region=eu-west-1

How to reproduce

$email = (new TemplatedEmail())
            ->from(new Address('emailFROM@email.com', 'nameFROM'))
            ->to(new Address('emailTO@email.com', 'nameTO'))
            ->bcc(new Address('emailBCC@email.com', 'nameBCC'))
            ->subject($subject)
            ->htmlTemplate($template)
            ->context($context);

$mailer->send($email);

• Email received at emailTO@email.com
• Email not received at emailBCC@email.com

Nothing more than that. Upon debugging, it seems that when the message is converted to raw the bcc is not present in the headers.

[FlaverSaver]

The same problem also occurs with SendmailTransport. The cause can be found here: https://github.com/symfony/mime/blob/2aad3728327b3b1fd8137004528129673dec1e09/Message.php#L96
The BCC is removed in all cases.

[Pixelshaped]

The same problem also occurs with SendmailTransport. The cause can be found here: https://github.com/symfony/mime/blob/2aad3728327b3b1fd8137004528129673dec1e09/Message.php#L96
The BCC is removed in all cases.

The commentary really makes it look like it's a voluntary omission. Is there any other way to add BCCs ? The issue made me double my API calls (not a big deal in my case but still)

[hyde1]

Hello, same problem here.
SesApiTransport is reading recipients from the Enveloppe when sending messages without attachement.
But SesHttpTransport and SesApiTransport with not attachement are both sending the raw message and without reading the Enveloppe, where all recipients are present...
I think it should use the Destination parameter of the SendRawEmail action by reading the Eveloppe recipients before sending the request.

[akondas]

The problem is with AbstractHttpTransport which uses SentMessage class to send e-mail. All comes down to one method: Symfony\Component\Mime\Message::getPreparedHeaders

        // remove the Bcc field which should NOT be part of the sent message
        $headers->remove('Bcc');

This should be changed to use Email class like AbstractApiTransport, I could fix it, but I don't know if we can do it this way 🤔, maybe I will suggest PR to show you what I mean?

[joey-bolts]

Same issue here, all Bcc's are removed and not send. Any status update?

[fabpot]

I'm not using AWS SES, so someone needs to investigate this issue and submit a patch to fix it. I will then more than happy to review it and merge it.

[frans-beech-it]

Did some more digging.

The BCC is removed from the header in general for all transports here https://github.com/ehymel/symfony/blob/35b7e1cb9a9c3fafe0c68386fae13721ea05aa82/src/Symfony/Component/Mime/Message.php#L99

But when using sendmail the bcc header is needed and stripped by sendmail for instance.

If we look at the swiftmail implementation you see BCC is only removed for SMTP see https://github.com/swiftmailer/swiftmailer/blob/59935da5e9d4c7ae801bb393d638dac2515498c9/lib/classes/Swift/Transport/AbstractSmtpTransport.php#L203

@fabpot would that also be a proper solution for symfony/mailer?

[jderusse]

SES issue will be fixed by #39743 You can use ses+api as a workaround. Or bump to symfony 5.2 + install async-aws/ses

[jderusse]

Regarding the sendmail issue, could somebody test this PR #39744 ?

/cc @frans-beech-it

[frans-beech-it]

@nicolas-grekas @jderusse should we create a new issue for the sendmail transport issue?