geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hendrik Saly (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (GERONIMO-6533) [Geronimo Javamail] InternetAddress should throw exception in strict mode for invalid address
Date Sat, 18 Oct 2014 22:33:33 GMT

    [ https://issues.apache.org/jira/browse/GERONIMO-6533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14176155#comment-14176155
] 

Hendrik Saly commented on GERONIMO-6533:
----------------------------------------

Here is proposal for fixing this: https://github.com/salyh/javamailspec/commit/97bd36001ce560c31f241fde37b0ee37732def6e

Although the above mentioned fix proposal fixes the thing in a spec compliant manner its to
be considered as a "breaking" change because it can make stop working applications which delivering
mail to local addresses.
So far a internet address with only a local part ("a" for example) was considered as correct
even in strict mode (which is not rfc822 compliant because a domain is also needed). Now such
an address will be considered as illegal in strict mode which can be breaking.

Comments? 

> [Geronimo Javamail] InternetAddress should throw exception in strict mode for invalid
address
> ---------------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-6533
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-6533
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: mail
>            Reporter: Arnaud MERGEY
>
> Unlike Oracle Javamail to Geronimo Javamail implementation, Geronimo Javamail accept
invalid emails even in strict mode
> Executing;
> <code> javax.mail.internet.InternetAddress("a",true);</code>
> fails throwing AddressException with Oracle Javamail, but succeed with Geronimo Javamail
> validate() succeed as well, it should throw an exception when address is not an email
address



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message