Continuum

multiple email addresses delimited with commas causes AddressException

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.2
  • Component/s: Notifier - Mail
  • Labels:
    None
  • Complexity:
    Intermediate
  • Number of attachments :
    0

Description

Email notification fails with a "javax.mail.internet.AddressException: Illegal route-addr in string" exception when a comma-delimited list of email addresses is used.

See the linked message for the pom.xml node for an example.
There is a work-around, see this thread in user list

http://mail-archives.apache.org/mod_mbox/maven-continuum-users/200609.mbox/browser

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Emmanuel Venisse added a comment -

Already fixed.

Show
Emmanuel Venisse added a comment - Already fixed.
Hide
Permalink
Mat Schaffer added a comment -

Where was this fixed? I'm running 1.1 and still getting:

Caused by: javax.mail.internet.AddressException: Illegal route-addr in string ``...@...com, ...@...com, ...@...com''

I can't find anything in the logs at http://svn.apache.org/repos/asf/maven/continuum/trunk that refers to it. I'm still hunting in http://svn.apache.org/repos/asf/maven/continuum

Thanks!

Show
Mat Schaffer added a comment - Where was this fixed? I'm running 1.1 and still getting: Caused by: javax.mail.internet.AddressException: Illegal route-addr in string ``...@...com, ...@...com, ...@...com'' I can't find anything in the logs at http://svn.apache.org/repos/asf/maven/continuum/trunk that refers to it. I'm still hunting in http://svn.apache.org/repos/asf/maven/continuum Thanks!
Hide
Permalink
Paul Benedict added a comment -

Same here. Running 1.1 and still have this error.

Show
Paul Benedict added a comment - Same here. Running 1.1 and still have this error.
Hide
Permalink
Wendy Smoak added a comment -

Reopening based on comments.

Show
Wendy Smoak added a comment - Reopening based on comments.
Hide
Permalink
Olivier Lamy added a comment -

fixed in rev 647001.

Show
Olivier Lamy added a comment - fixed in rev 647001.
Hide
Permalink
Mat Schaffer added a comment -

I dug up the diff at http://svn.apache.org/viewvc/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/notification/mail/MailContinuumNotifier.java?r1=647001&r2=647000&pathrev=647001

According to this the fix only suppresses the error by splitting on comma and sending the email to the first address on the list only. This isn't the behavior I was hoping for. Paul may have similar feelings.

Thanks for the fix though. This at least gets rid of the error. I may open another ticket to explore support for sending multiple email addresses from a single notifier.

Show
Mat Schaffer added a comment - I dug up the diff at http://svn.apache.org/viewvc/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/notification/mail/MailContinuumNotifier.java?r1=647001&r2=647000&pathrev=647001 According to this the fix only suppresses the error by splitting on comma and sending the email to the first address on the list only. This isn't the behavior I was hoping for. Paul may have similar feelings. Thanks for the fix though. This at least gets rid of the error. I may open another ticket to explore support for sending multiple email addresses from a single notifier.
Hide
Permalink
Olivier Lamy added a comment -

yes please

Show
Olivier Lamy added a comment - yes please

People

Vote (3)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Codehaus. Try JIRA - bug tracking software for your team.