Redmine

Sorry the first patch is wrong

RFC2821:
When the delivery SMTP server makes the "final delivery" of a message, it inserts a return-path line at the beginning of the mail data.

http://stackoverflow.com/questions/1235534/what-is-the-behavior-difference-between-return-path-reply-to-and-from:
The Return-Path (sometimes called the Reverse-Path or Envelope-FROM -- all of these terms can be used interchangeably) is the value used during the SMTP session. As you can see, this does not need to be the same value that is actually found in the mail headers. Only the recipient's mail server is supposed to add a Return-Path header to the top of the email. This records the actual Return-Path sender during the SMTP session. If a Return-Path header is already exists in the email, then that header is to be removed, and replaced by the recipient's mail server.

Reading this, I don't think we need to implement this, what do you think?

Hi again, I had to add a Return-Path header in emails send by a Redmine instance.
I think that only the sender can determine a correct (non default) Return-Path.
The case is the following :
E-mails are send with the default email address : admin@yourcompany.com
But we want that the real user initiating the action receives the bounce emails, because he will be the one knowing how to fix wrong recipients emails (and not the application adminstrators that have other cats to whip :-) ).
We use this and it works perfectly well with exim on the application server and zimbra as the final company mail server.

We now override the Mailer model initialize method

with :

         def initialize_with_return_path(*args)
            initialize_without_return_path(*args)

            # Specific : Return-path changed for the user making the change
            # Only for company members (*@[country_iso_code.]smile[-<country>].[extension])
            if (
              User.current.mail.present? &&
              (
                # A regexp to identify your company users by their email
                # You can use something else
                /@([a-z]+\.)?smile(-[a-zA-Z1-9]+|)?\.[a-z]+$/.match(User.current.mail)
              )
            )
              headers['Return-Path'] = User.current.mail
              # Rails.logger.debug "==>mail Return-Path=#{headers['Return-Path']} (#{User.current.mail})" 
            end
            # END -- Specific
          end

Hi, you can indicate in the Resolution that's it Won't be Fixed, if you have the time to, thanks a lot.

Jérôme BATAILLE wrote:

Hi, you can indicate in the Resolution that's it Won't be Fixed, if you have the time to, thanks a lot.

"Patch" tracker does not have "Resolution" filed.