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

Empty emailAddress causes Mailer error

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Component/s: core
    • Labels:
      None
    • Similar Issues:

      Description

      When users delete their email in user profile, the configuration file leaves an empty node in XML user configuration file

      <hudson.tasks.Mailer_-UserProperty>
      <emailAddress></emailAddress>
      </hudson.tasks.Mailer_-UserProperty>

      Jenkins then behaves like described here https://issues.jenkins-ci.org/browse/JENKINS-5625

      Can you change the hudson.tasks.Mailer.UserProperty#getAddress() implementation to the code sguuested below and this bug will be fixed:

      @Exported
      public String getAddress()

      { if (emailAddress != null && emailAddress.trim().length() > 0) return emailAddress; // try the inference logic return MailAddressResolver.resolve(user); }

      This patch checks that email setting in user profile, and ensures that empty string/setting behaves like there is no setting present and takes generates the correct email address according to valid security realm implementation (if such) or appends default suffix (if deffined).

      Thanks.

        Attachments

          Issue Links

            Activity

            mstipanov Marko Stipanov created issue -
            sogabe sogabe made changes -
            Field Original Value New Value
            Assignee sogabe [ sogabe ]
            scm_issue_link SCM/JIRA link daemon made changes -
            Resolution Fixed [ 1 ]
            Status Open [ 1 ] Resolved [ 5 ]
            evernat evernat made changes -
            Link This issue is duplicated by JENKINS-10296 [ JENKINS-10296 ]
            rtyler R. Tyler Croy made changes -
            Workflow JNJira [ 140491 ] JNJira + In-Review [ 189082 ]

              People

              Assignee:
              sogabe sogabe
              Reporter:
              mstipanov Marko Stipanov
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: