camel-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "slad (JIRA)" <j...@apache.org>
Subject [jira] Commented: (CAMEL-2067) Mail Component: Character "|" in recipient addresses causes failure
Date Fri, 16 Oct 2009 22:25:53 GMT

    [ https://issues.apache.org/activemq/browse/CAMEL-2067?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=54801#action_54801
] 

slad commented on CAMEL-2067:
-----------------------------

Thanks! I agree that having comma or pipe in email is pretty rare. But comma or semi-colon
can be part of name. But we can put some validation on our side.

> Mail Component: Character "|" in recipient addresses causes failure
> -------------------------------------------------------------------
>
>                 Key: CAMEL-2067
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-2067
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-mail
>    Affects Versions: 1.6.1, 2.0.0
>            Reporter: slad
>            Assignee: Claus Ibsen
>            Priority: Trivial
>             Fix For: 1.6.2, 2.1.0
>
>
> It looks like there is minor bug in regex used in MailBinding.java, it causes "|" to
act as delimiter, splitting email address into 2 parts.  
> For ex: Using email address as - first|last@test.com causes it to split into two parts:
> 1) first
> 2) last@test.com
> "|" is acceptable character in email as per wikipedia [http://en.wikipedia.org/wiki/Email_address].
I did not bother to read RFC specification.
> Here is snippet of code causing issue:
>  private static void appendRecipientToMimeMessage(MimeMessage mimeMessage, String type,
String recipient)
>         throws MessagingException {
>         // we support that multi recipient can be given as a string seperated by comma
or semi colon
>         String[] lines = recipient.split("[,|;]");
>         for (String line : lines) {
> Regex above should either be "[,;]" or ";|,"
> Also, having email addresses separate out by , or ; seems to be issue as it can be part
of name. ex: "firstName, I like comma <first.last@test.com>" is valid. Above code will
still cause failure for such messages. Can camel support addresses as List/Array rather than
comma delimited string?
> See Forum thread: http://www.nabble.com/Camel-Mail-Component%3A-%22|%22-in-email-causes-message-to-fail-td25883099.html

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message