[GitHub] [incubator-ponymail] sebbASF commented on issue #502: Bug: parser fails to detect inline attachments

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

sebbASF commented on issue #502: Bug: parser fails to detect inline attachments
URL: https://github.com/apache/incubator-ponymail/issues/502#issuecomment-520233834
 
 
   The archiver code currently walks the message parts twice: in msgbody and msgfiles.
   
   This makes it harder to ensure all parts are accounted for exactly once.
   
   At present the methods each look for specific headers, which is how some parts have been missed.
   
   There will need to be some co-ordination between the two methods to avoid omissions and duplications (there's no point creating an attachment record for the main body).
   
   Alternatively, the parts could be walked once and allocated as body or attachment.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services