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

Patch to be able to use mail server submission port (587)

XMLWordPrintable

    • Icon: Patch Patch
    • Resolution: Duplicate
    • Icon: Major Major
    • mailer-plugin
    • None
    • Platform: All, OS: All

      A check box in the email configuration tool is now available to use the
      submission port instead of the default port. The patch is using code similar to
      the "Use SSL" check box.

        1. submission_port_patch.txt
          7 kB

          [JENKINS-1756] Patch to be able to use mail server submission port (587)

          xlv added a comment -

          Created an attachment (id=268)
          submission port patch

          xlv added a comment - Created an attachment (id=268) submission port patch

          Kohsuke Kawaguchi added a comment -

          Thank you for the patch.
          But given issue #984, isn't it better to just let the user enter the port number?

          Any chance you can revise a patch along that line?

          Kohsuke Kawaguchi added a comment - Thank you for the patch. But given issue #984, isn't it better to just let the user enter the port number? Any chance you can revise a patch along that line?

          xlv added a comment -

          I saw the other issue but I figured the submission port was well defined and
          that's what I use on my mail server. I was going to submit a patch for the other
          issue later too.

          I can modify my patch to use a "Use Custom Port" instead and people using the
          submission port will just need to enter 587 in the field. I'll add that port in
          the documentation as an example (unless you want to have both options, numeric
          port and checkbox).

          xlv added a comment - I saw the other issue but I figured the submission port was well defined and that's what I use on my mail server. I was going to submit a patch for the other issue later too. I can modify my patch to use a "Use Custom Port" instead and people using the submission port will just need to enter 587 in the field. I'll add that port in the documentation as an example (unless you want to have both options, numeric port and checkbox).

          Kohsuke Kawaguchi added a comment -

          Yes, sounds like a plan.

          Thank you!!

          Kohsuke Kawaguchi added a comment - Yes, sounds like a plan. Thank you!!

          xlv added a comment -

          I've submitted a patch for <a href="show_bug.cgi?id=984">issue #984, </a> so
          this one could be closed unless you want me to adapt it to also have the "Use
          Submission Port" option.

          xlv added a comment - I've submitted a patch for <a href="show_bug.cgi?id=984">issue #984, </a> so this one could be closed unless you want me to adapt it to also have the "Use Submission Port" option.

          Kohsuke Kawaguchi added a comment -

          Yep. Marking as a duplicate for #984.

              • This issue has been marked as a duplicate of 984 ***

          Kohsuke Kawaguchi added a comment - Yep. Marking as a duplicate for #984. This issue has been marked as a duplicate of 984 ***

            Unassigned Unassigned
            xlv xlv
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved: