Issue Details (XML | Word | Printable)

Key: CONTINUUM-2281
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: Jose Morales Martinez
Reporter: Klaus Wienert
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
Continuum

Same Recipiant will be added several times in Notification

Created: 29/Jun/09 10:32 AM   Updated: 04/Aug/09 10:12 AM   Resolved: 04/Aug/09 10:12 AM
Return to search
Component/s: Core system, Notifier - Mail
Affects Version/s: 1.3.3, 1.3.4
Fix Version/s: 1.3.4

Time Tracking:
Not Specified

File Attachments: None
Image Attachments:

1. 2009-06-29_171922.png
(41 kB)
Environment: Windows, Tomcat, JDK 5, Subversion

Complexity: Intermediate


 Description  « Hide

If notifier configuration is committers = true, then for each SCM change the Author is added in To Address. Even if the Author is already a Recipient. The Author will only get one email, but the To Address will get right long.

http://svn.apache.org/repos/asf/continuum/tags/continuum-1.3.3/continuum-core/src/main/java/org/apache/maven/continuum/notification/mail/MailContinuumNotifier.java

for ( ChangeSet changeSet : changes )
{
String scmId = changeSet.getAuthor();
if ( StringUtils.isNotEmpty( scmId ) )
{
String email = developerToEmailMap.get( scmId );
if ( StringUtils.isEmpty( email ) )

{ //TODO: Add a default domain so mail address won't be required log.warn( "no email address is defined in developers list for '" + scmId + "' scm id." ); }

else

{ // TODO: set a proper name InternetAddress to = new InternetAddress( email.trim() ); log.info( "Recipient: To '" + to + "'." ); message.addRecipient( Message.RecipientType.TO, to ); }

}
}

<ciManagement>
<system>continuum</system>
<url>http://server/continuum/</url>
<notifiers>
<notifier>
<type>mail</type>
<sendOnError>true</sendOnError>
<sendOnFailure>true</sendOnFailure>
<sendOnSuccess>false</sendOnSuccess>
<sendOnWarning>true</sendOnWarning>
<configuration>
<committers>true</committers>
</configuration>
</notifier>
</notifiers>
</ciManagement>



Jose Morales Martinez added a comment - 04/Aug/09 10:12 AM

Fixex in 800837r for version 1.4.0.
Fixed in 800811r for version 1.3.4.