When Webform is configured to use a Reply-To header for messages the email that gets sent has no Reply-To header set.

CommentFileSizeAuthor
#2 sendgrid_integration-3014573-2.patch692 bytesaveragejoe3000

Comments

Perignon created an issue. See original summary.

averagejoe3000’s picture

averagejoe3000
he/him
Primary language English
Location Swanzey, NH
commented
StatusFileSize
new sendgrid_integration-3014573-2.patch692 bytes

Sendgrid Integration is checking for case permutations of 'reply-to', but only checks for 'reply-to' and 'Reply-To'. Webform is setting the message header key as 'Reply-to', so Sendgrid Integration never sets the reply to on the email.

Instead of trying to check for possible case permutations of 'Reply-to' and then assigning it to a new array key, use the reply to key in the setReplyTo.

perignon’s picture

perignon commented

This is such an annoying problem that has come up before. It is the game of whack-a-mole trying to accommodate for spelling permutations of email headers. Wish we had a standard across all Drupal.

averagejoe3000’s picture

averagejoe3000
he/him
Primary language English
Location Swanzey, NH
commented
Status: Active » Needs review

I've now used my patch for a few weeks on a production site, and have not run into any issues.

perignon’s picture

perignon commented

Thanks for the patch. I pushed it into Dev. I got to backport this change to 7.x

perignon’s picture

perignon commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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