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

Empty emailAddress causes Mailer error.

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved (View Workflow)
    • Priority: Major
    • Resolution: Duplicate
    • Component/s: None
    • 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

            Hide
            slide_o_mix Alex Earl added a comment -

            It would be useful if you wanted to fork the project on github and then submit a pull request if you believe your fix will work.

            Show
            slide_o_mix Alex Earl added a comment - It would be useful if you wanted to fork the project on github and then submit a pull request if you believe your fix will work.

              People

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

                Dates

                Created:
                Updated:
                Resolved: