[jira] [Created] (JAMES-3712) Bounce does not add prefix to the sent email

["Benoit Tellier (Jira)" <se...@james.apache.org>]

Benoit Tellier created JAMES-3712:
-------------------------------------

             Summary: Bounce does not add prefix to the sent email
                 Key: JAMES-3712
                 URL: https://issues.apache.org/jira/browse/JAMES-3712
             Project: James Server
          Issue Type: Bug
          Components: Mailet Contributions
            Reporter: Benoit Tellier


Reported by @ljose on gitter

While upgrading from 2.6 to 3.6 he noticed a breaking change to Bounce mailet, not adding the prefix to the sent email, and instead adding it to the email in flight.

To quote him "It looks like it is prefixed with "Undeliverable" after Bounce, but when it comes in as a new incoming mail, the subject is without the prefix."

CF https://gitter.im/apache/james-project?at=61fb3bbe3349fe1c71f18823

1. Its friend, DSNBounce does modify the sent message, not the in flight one (what is described above)

2. This seems logical that DSNBounce modifies the sent email: itself knows how to modify it. The in flight message could still be modified by follow up mailets

3. The refactoring of Bounce* classes was done with an emphase on preserving behaviour, without questionning it - what I am doing here.  To be fairly honnest I bet on a copy-and-paste mistake that was never questionned, and duplicated to other classes.

After a short review, a few other mailets are impacted too: NotifySender + NotifyPostmaster...



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org