[GitHub] [james-project] chibenwa commented on pull request #730: JAMES-3666 Fix DSNBounce exception when no Date header is present

[GitBox <gi...@apache.org>]

chibenwa commented on pull request #730:
URL: https://github.com/apache/james-project/pull/730#issuecomment-958656969


   > Wondering why there is twice the exact same commit though?
   
   My plan was to squash it all upon merge ;-)
   
   > There is actually a unit test for this, but the MimeMessageBuilder adds a Date header by default if not specified. The test fix is to explicitly remove the Date header after building.
   
   We might want to audit the code base, and MIME related tests regarding this defect. If it happened one time, we might have used a similar pattern in other places of the code and felt safe...


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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