Missing support for header Reply-To

Created on 12 April 2024, 8 months ago

Updated 26 April 2024, 7 months ago

Problem/Motivation

After configuring the module, I got the error

Error sending email (from xxxxxxxxxxxxxx to xxxxxxxxxxxxx with reply-to not set).

I am using sendgrid and the reply to is mandatory. So naturally, I change my hook_mail to set the header Reply-To but I keep getting the same error.
After a little bit of digging, I noticed that this module just ignore the Reply-To header and will not set any Reply-To unless there is a "Sender" header.

Please review patch attached.

After applying the patch, the test email has been sent without any issue.

Steps to reproduce

Proposed resolution

Just simply add support to set the reply to if a "Reply-To" header is detected.

Remaining tasks

User interface changes

API changes

Data model changes

✨ Feature request

Status

Fixed

Version

1.0

Component

Code

Created by

🇨🇦Canada phjou Vancouver 🇨🇦 🇪🇺

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @phjou
Comment 8 months ago →
🇨🇦Canada phjou Vancouver 🇨🇦 🇪🇺
Status changed to Needs review 8 months ago6:45am 12 April 2024
Comment 8 months ago →
🇨🇦Canada phjou Vancouver 🇨🇦 🇪🇺

Comment 7 months ago →

System Message

zengenuity → committed 26818413 on 1.0.x

Issue #3440409 by phjou, zengenuity: Missing support for header Reply-To

Status changed to Fixed 7 months ago2:27pm 12 April 2024
Comment 7 months ago →
🇺🇸United States zengenuity
Looks good. Committed. Thanks!
Comment 7 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024