sbp commented on pull request #20: URL: https://github.com/apache/incubator-ponymail-foal/pull/20#issuecomment-835372590
The `original_content` attribute preserves the message delivered from the MTA to Mailman3, before Mailman3 potentially modifies the message. This is necessary because the potential modifications made by Mailman3 could affect the DKIM-ID. But it may be possible to implement deduplication in a better way than DKIM-ID, by manipulating the message database *post hoc*, either in `archiver.py` or using a background task or both. This would mean that `original_content` would not be necessary, at least for Mailman3: `msg.to_bytes()` or `bytes(msg.as_string(), "utf-8")` could be used instead. I'm going to close the present PR to make way for a new one using this alternative approach. -- 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. For queries about this service, please contact Infrastructure at: [email protected]
