Hi together

I recognized, that mail safety sent out mails to BCC and CC addresses, when "send_mail_to_default_mail" is enabled AND the property is spelled different in mail_safety_mail_alter's $message['headers'] array.

So the code removes only properties BCC and CC, but not Bcc or Cc. I recognized that, when commerce's receipt Bcc mailwas sent out and was not catched by mail_safety.

Will prepare a patch for that.

CommentFileSizeAuthor
#5 3046820-5.patch511 bytesandreasderijcke
#2 3046820-2-remove-bcc-and-cc-case-insensitive.patch754 bytesmegadesk3000

Comments

megadesk3000 created an issue. See original summary.

megadesk3000’s picture

megadesk3000 commented
StatusFileSize
new 3046820-2-remove-bcc-and-cc-case-insensitive.patch754 bytes

Patch created

megadesk3000’s picture

megadesk3000 commented
Assigned: megadesk3000 » Unassigned
Status: Needs work » Needs review
andreasderijcke’s picture

andreasderijcke
Location Antwerpen / Gent
commented
Status: Needs review » Reviewed & tested by the community

Works for me, both on 8.x-1.0 as on 8.x-1.x#0664357f

andreasderijcke’s picture

andreasderijcke
Location Antwerpen / Gent
commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new 3046820-5.patch511 bytes

Actually, as stated in issue #3073570, I think 'Cc' and 'Bcc' is the only correct notation.
That change the patch for this issue as attached.

  • Nitebreed committed c2f3576 on 8.x-1.x 
    Issue #3046820 by megadesk3000, andreasderijcke: BCC and CC not removed...
nitebreed’s picture

nitebreed
Primary language Dutch
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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