Hi @kbond, you implemented the original version for Mailgun TagHeader support, are you the right person to ask for a review/feedback?

Thanks for your reply! Any change I can convince you to handle this as a bug? 😄
Main reason is that multiple TagHeader does work with the MailgunHttpTransport and this PR fixes it for MailgunApiTransport.

I admit this feels like a bit of a grey area personally and I know the frustration of having to wait a few versions for a fix like this. I'm falling back on the established Symfony process and this would be considered a feature. The code isn't currently broken, it just doesn't allow a specific feature provided by Mailgun.

Oke fair enough, it was worth a try. I will rebase and target 6.1 :)

Awesome, and thanks for understanding :)

Thank you @starred-gijs.

@OskarStark Thanks for the review!
Sorry did not read your comment in time to change it, its now already merged so I will leave it as is.

No problem, thanks for reaching out

I've made the change myself while merging as it was a small one.

I admit this feels like a bit of a grey area personally and I know the frustration of having to wait a few versions for a fix like this. I'm falling back on the established Symfony process and this would be considered a feature. The code isn't currently broken, it just doesn't allow a specific feature provided by Mailgun.

Not to disrespect anyone, but how is this PR different? #45274

edit: I mean the target branch :)

Not to disrespect anyone

None taken :)

but how is this PR different?

I chose #45274 as a bug fix because multiple tags were already implemented, tested, and documented for Mailchimp. The way they were added was not consistent.