Uploaded image for project: 'Continuum'
  1. Continuum
  2. CONTINUUM-2281

Same Recipiant will be added several times in Notification

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 1.3.3 (Beta), 1.3.4 (Beta)
    • 1.3.4 (Beta)
    • None
    • Windows, Tomcat, JDK 5, Subversion

    Description

      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>

      Attachments

        1. 2009-06-29_171922.png
          41 kB
          Klaus Wienert

        Activity

          People

            jomm Jose Morales Martinez
            klaus.wienert Klaus Wienert
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Issue deployment