[JAMES-3415] MailEnvelopeImpl should use a DeferedInputStream

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.6.0
• Component/s: SMTPServer
• Labels: bug easy-fix
• Assignee: Unassigned
• Reporter: Benoit Tellier
• Created: 06/Oct/20 08:53
• Updated: 08/Mar/21 06:47
• Resolved: 08/Mar/21 06:47

Description

Following remarks made by @adarshasp87 on gitter,

it turns out MailEnvelopeImpl uses an inmemory data structure to store temporary messages before enqueuing them.

When the size extension is not enabled, this can cause "out of memory exceptions".

We would rather like to rely on a DeferedInputStream, or the like, in order to allow very large messages reception in SMTP.

Community members should feel free to contribute this simple fix.