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

Invalid cc:, bcc: recipients handling when mail address contains a dot

    Details

    • Similar Issues:

      Description

      I have a strange behaviour with e-mail addresses having dot. Presumably Ext plugin has a typo in inet-address parsing in ListProvider.

      Case 1:
      Project Recipient List: adonai.elohim, cc:master.maker, bcc:lich.king
      In this case Ext plugin only sends E-mail to adonai.elohim$suffix
      CC and BCC are ignored

      Case 2:
      Project Recipient List: adonai.elohim, cc:"master.maker", bcc:lich.king
      In this case Ext plugin sends E-mail to adonai.elohim$suffix
      and CC's to "master.maker"$suffix,
      BCC is ignored
      (btw I see these quotes around names when reading with mail-client)

      Case 3:
      Project Recipient List: adonai.elohim, cc:"master.maker", bcc:"lich.king"
      In this case Ext plugin sends E-mail to adonai.elohim$suffix
      and CC's to "master.maker"$suffix and "lich.king"$suffix,
      no BCC is sent, all recipients are visible

        Attachments

          Activity

          Hide
          adonai Adonai Elohim added a comment -

          Ok, I'll go take a look at DavMail source. Don't close this issue, I'll update it with non-quoted version soon

          Show
          adonai Adonai Elohim added a comment - Ok, I'll go take a look at DavMail source. Don't close this issue, I'll update it with non-quoted version soon
          Hide
          slide_o_mix Alex Earl added a comment -

          I won't close until resolved.

          Show
          slide_o_mix Alex Earl added a comment - I won't close until resolved.
          Hide
          slide_o_mix Alex Earl added a comment -

          This actually has nothing to do with dots in the email address. It has to do with a bug I introduced that parsed the cc and bcc stuff incorrectly. I was not seeing it on the code I was using because I had already fixed it. I should have run earlier with the released version of the code. Should have a fix in the next version.

          Show
          slide_o_mix Alex Earl added a comment - This actually has nothing to do with dots in the email address. It has to do with a bug I introduced that parsed the cc and bcc stuff incorrectly. I was not seeing it on the code I was using because I had already fixed it. I should have run earlier with the released version of the code. Should have a fix in the next version.
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Alex Earl
          Path:
          src/main/java/hudson/plugins/emailext/EmailRecipientUtils.java
          http://jenkins-ci.org/commit/email-ext-plugin/cea631c421b7964ba710d21c3a4131d7a881f739
          Log:
          Fix JENKINS-28444

          Fix incorrect check for > 0 size on the bcc an cc lists.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Alex Earl Path: src/main/java/hudson/plugins/emailext/EmailRecipientUtils.java http://jenkins-ci.org/commit/email-ext-plugin/cea631c421b7964ba710d21c3a4131d7a881f739 Log: Fix JENKINS-28444 Fix incorrect check for > 0 size on the bcc an cc lists.
          Hide
          adonai Adonai Elohim added a comment -

          Oh!
          Very sorry for not providing logs.
          Suddenly got very sick.

          Thanks for sorting this out anyways!

          Show
          adonai Adonai Elohim added a comment - Oh! Very sorry for not providing logs. Suddenly got very sick. Thanks for sorting this out anyways!

            People

            • Assignee:
              slide_o_mix Alex Earl
              Reporter:
              adonai Adonai Elohim
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: