search

kirviq / dumbster

Dumbster - a simple fake SMTP server for unit testing
Apache License 2.0
70 stars 39 forks source link

Data with \r\n\t not handled by input scanner \r\n #9

Open camboui opened 2 years ago

camboui commented 2 years ago

Hi,

I found at that javax.mail.internet, is splitting InternetAddress lists longer than 76 characters with character "\r\n\t". See line 503 of https://github.com/javaee/javamail/blob/master/mail/src/main/java/javax/mail/internet/InternetAddress.java

I guess it's standart. However, it seems that dumbster doesn't handle it yet.

STEP TO REPRODUCE : Send an email with Header "To" value set to "mail1@mail.com, email1@email.com, \r\n\tAnymail@email.com"

BEHAVIOUR email.getHeaderValue("To"); equals "mail1@mail.com, email1@email.com,"

EXPECTED BEHAVIOUR email.getHeaderValue("To"); equals "mail1@mail.com, email1@email.com, Anymail@email.com"

PROBABLE CAUSE Scanner reading data line by line with"\r\n" pattern. It leaves an "\t" which then prevent com.dumbster.smtp.SmtpMessage.java.store() to append header value.

Thanks, Hope this post will at least make people save some time 😄