SpartaSystems / holdmail

HoldMail is a Java Application for proxying SMTP mail, providing a browsable viewer for captured emails.
Apache License 2.0
32 stars 15 forks source link

Issue #14 - Fix /rest/messages/ID/raw #41

Closed barryoneill closed 6 years ago

barryoneill commented 6 years ago

Short description of what this resolves:

With commit 810eb22, the 'messageRaw' attribute was deprecated and toggled to return 'null' by default in https://github.com/SpartaSystems/holdmail/blob/master/src/main/java/com/spartasystems/holdmail/mapper/MessageSummaryMapper.java#L66

This was because large attachments in an email would cause the response for message summary to be huge, and would cause delays in the UI. Now the UI must additionally call /rest/messages/ID/raw if it wishes to fetch the original mail content.

However, I forgot to actually fix /rest/messages/ID/raw and it was returning 404 🙄

Changes proposed in this pull request: