Code

Opened 7 years ago

Closed 7 years ago

Last modified 2 years ago

#5746 closed Uncategorized (wontfix)

Patch to add support for setting email return-path

Reported by: wreese@… Owned by: nobody
Component: Core (Mail) Version: master
Severity: Normal Keywords: return-path email mail
Cc: Triage Stage: Design decision needed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Our Django application needs to send customized mail messages using
Variable Envelope Return Paths (VERPs - http://en.wikipedia.org/wiki/VERP ).
This simply means setting the return path of each outbound
email message to something unique, so that bounced messages can be
accurately tracked and handled.

In order to do this, developers need to be able to set the
"from_addr" parameter that is passed through to the sendmail() method
of the smtplib.SMTP class. Currently, Django automatically sets this
parameter based on the "From" header that is set on the EmailMessage
object.

The attached patch simply modifies the EmailMessage and
SMTPConnection classes to allow developers to manually set the return
path attribute, while safely defaulting to the "From" header (the
current default) if it is not supplied. The changes are minimal,
and tests and documentation have been written and supplied with the patch.

Attachments (1)

return_path.diff (6.7 KB) - added by wreese@… 7 years ago.
Patch to add support for setting email return-path

Download all attachments as: .zip

Change History (5)

Changed 7 years ago by wreese@…

Patch to add support for setting email return-path

comment:1 Changed 7 years ago by mtredinnick

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

At the moment, I'm -1 on this. We made EmailMessage and SMTPConnection subclassable for exactly this reason: so that people with specialised requirements can make their own modifications without needing us to change the core.

Leaving open for a bit to think about it.

comment:2 Changed 7 years ago by mtredinnick

  • Triage Stage changed from Unreviewed to Design decision needed

comment:3 Changed 7 years ago by jacob

  • Resolution set to wontfix
  • Status changed from new to closed

I agree with Malcolm, so I'm marking this wontfox.

comment:4 Changed 2 years ago by guettli

  • Easy pickings unset
  • Severity set to Normal
  • Type set to Uncategorized
  • UI/UX unset

related #9214

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.