Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-7156

Incorrect mail address used for sending email notifications

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      Incorrect email address is used for Git commits when sending email notifications, this can be seen in the console log for a failed build:

      Hudson Console:
      ---------------
      (...)
      Waiting for Hudson to finish collecting data
      ERROR: Illegal whitespace in address
      channel stopped
      ERROR: Illegal whitespace in address
      javax.mail.internet.AddressException: Illegal whitespace in address in string ``Geir Pettersen@comperio.no''

      Notice how the author's full name is used before the @ instead of the account name.
      By executing "git whatchanged" I can verify that the commit contains a correct email address in the author field:

      $ git whatchanged
      commit 53f56784f7299465e33f0cc483a397f23ea59fe5
      Author: Geir Gullestad Pettersen <ggp@comperio.no>
      Date: Mon Aug 9 02:03:35 2010 +0200

      hudson test...

      :100644 100644 5d2a1e7... fba468b... M src/main/java/no....

        Attachments

          Issue Links

            Activity

            Hide
            ge0ffrey Geoffrey De Smet added a comment -

            abayer accepted my pull request with the changes proposed by rogerhu
            https://github.com/jenkinsci/git-plugin/commit/8364a5bf6c26c9901e3ffbc19f9aa7bda4bd65a4

            I presume this issue is now fixed.

            Show
            ge0ffrey Geoffrey De Smet added a comment - abayer accepted my pull request with the changes proposed by rogerhu https://github.com/jenkinsci/git-plugin/commit/8364a5bf6c26c9901e3ffbc19f9aa7bda4bd65a4 I presume this issue is now fixed.
            Hide
            david_beutel J. David Beutel added a comment -

            Maybe not. I just tried this snapshot from the trunk, and while it stopped sending email to the full name, it seems to be resolving the addresses to nothing now. The author lines in changelog.xml are in the proper format with full name, email, timestamp, and TZ. The user/Full Name/configure page also has the email address somehow, so even if that's overriding, it doesn't explain why the recipient list is empty.

            Tests FAILED - view reports in target/test-reports
            Application context shutting down...
            Application context shutdown.
            Recording test results
            An attempt to send an e-mail to empty list of recipients, ignored.

            Show
            david_beutel J. David Beutel added a comment - Maybe not. I just tried this snapshot from the trunk, and while it stopped sending email to the full name, it seems to be resolving the addresses to nothing now. The author lines in changelog.xml are in the proper format with full name, email, timestamp, and TZ. The user/Full Name/configure page also has the email address somehow, so even if that's overriding, it doesn't explain why the recipient list is empty. Tests FAILED - view reports in target/test-reports Application context shutting down... Application context shutdown. Recording test results An attempt to send an e-mail to empty list of recipients, ignored.
            Hide
            david_beutel J. David Beutel added a comment - - edited

            Never mind. It worked after I actually checked "Send separate e-mails to individuals who broke the build". Sorry.

            Show
            david_beutel J. David Beutel added a comment - - edited Never mind. It worked after I actually checked "Send separate e-mails to individuals who broke the build". Sorry.
            Hide
            slide_o_mix Alex Earl added a comment -

            There is still an issue with this where the git plugin somehow overrides the mailing address if there are git-svn repos. See JENKINS-13715

            Show
            slide_o_mix Alex Earl added a comment - There is still an issue with this where the git plugin somehow overrides the mailing address if there are git-svn repos. See JENKINS-13715
            Hide
            ndeloof Nicolas De Loof added a comment -

            git plugin now only set email address if not already set for User

            Show
            ndeloof Nicolas De Loof added a comment - git plugin now only set email address if not already set for User

              People

              • Assignee:
                ndeloof Nicolas De Loof
                Reporter:
                gpettersen gpettersen
              • Votes:
                17 Vote for this issue
                Watchers:
                18 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: