Bug: MimeMessageParser rejects attachments with duplicate names

[nicolasbelisle]

MimeMessageParser uses a Map for the attachments:

484 : public Map<String, DataSource> getAttachmentList()

However an email can have 2 attachments with the same name. The Map will just replace any existing attachment.

[bbottema]

Is this a real use case?

[nicolasbelisle]

Yes. Real email that went through MimeMessageParser (v. 5.1.7).

[nicolasbelisle]

Essentially replacing the Map with a List<Map.Entry) does the job
It's already called "attachmentList" ;-)
parsedComponents.attachmentList.put(cid, cidEntry.getValue());
to
parsedComponents.attachmentList.add(new AbstractMap.SimpleEntry(cid, cidEntry.getValue()));

BTW, you can add attachments with the same name in Gmail.
Reference nicolasbelisle@47a47f3 (There's a few other fixes in this commit however).

[bbottema]

Ok, you made a convincing case. If you feel up to it, I'll accept a PR on master! Or else I'll pick it up in due time.

[bbottema]

Fixed and released in 5.2.1!