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-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org
