bbottema/simple-java-mail

Bugfix: encoded delimited recipients in EML not parsed properly

fb-datax opened this issue · 7 comments

fb-datax commented

Using the following two eml Lines, the recipients list contains 2 entries instead of 1:

From: =?iso-8859-1?Q?Some_Name=2C_Jane_Doe?= <jane.doe@example.de>
To: =?iso-8859-1?Q?Some_Name_2=2C_John_Doe?= <john.doe@example.de>
Email email = EmailConverter.emlToEmail(eml);
System.out.println(email.getFromRecipient());
System.out.println(email.getRecipients());
Recipient{name='Some Name, Jane Doe', address='jane.doe@example.de', type=null}
Recipient{name='null', address='Some', type=To}, Recipient{name='John Doe', address='john.doe@example.de', type=To}]

If the "," is removed the recipients are parsed correctly:

From: =?iso-8859-1?Q?Some_Name_Jane_Doe?= <jane.doe@example.de>
To: =?iso-8859-1?Q?Some_Name_2_John_Doe?= <john.doe@example.de>
Recipient{name='Some Name Jane Doe', address='jane.doe@example.de', type=null}
[Recipient{name='Some Name 2 John Doe', address='john.doe@example.de', type=To}]

Problem occurs using the latest version 7.8.3. Using the 6.x Version the problem doesn't occur and the recipients are parsed correctly.

bbottema commented

If the "," is removed the recipients are parsed correctly:

I don't see a "," in your first example, can you double check please?

fb-datax commented

In the example above =2C -> ascii 44 -> ,

bbottema commented

ahh, I see. I'll have a look

bbottema commented

Any chance you can share an EML with me that has this issue?

bbottema commented

This was caused by the change for #293, which was released yesterday. I just released v7.9.1 that fixes this regression bug.

Thank you for reporting and for your prompt responses (and the test message)!

fb-datax commented

Thank you for fixing the issue so fast