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

Add SSL support for SMTP email

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Component/s: core
    • Labels:
      None
    • Environment:
      Platform: All, OS: All
    • Similar Issues:

      Description

      I've created a patch for using SSL sockets in the SMTP notification system.
      This is good for using gmail/google apps as your email server as my company
      does. I tried to honor the concept of allowing the end user to fully control
      the JavaMail smtp session by setting system properties. In fact, this is how I
      got this to originally work. What I did was add a "use SSL" checkbox on the
      main hudson email configuration page. If that checkbox is set, when an email is
      being sent it will set the smtp port to 465 and the socket factory to be the
      javax ssl socket factory. It does check to see if these are already set and if
      they are it does not override them. The idea is to give people a quick and easy
      way to enable ssl email but still allow advanced users full control.

        Attachments

        1. smtps.patch
          4 kB
          pfoorion
        2. smtp-use-ssl.html
          0.4 kB
          pfoorion

          Activity

          Hide
          pfoorion pfoorion added a comment -

          Created an attachment (id=44)
          patch/diff for Mailer.java and global.jelly

          Show
          pfoorion pfoorion added a comment - Created an attachment (id=44) patch/diff for Mailer.java and global.jelly
          Hide
          pfoorion pfoorion added a comment -

          Created an attachment (id=45)
          html help file for use ssl checkbox

          Show
          pfoorion pfoorion added a comment - Created an attachment (id=45) html help file for use ssl checkbox
          Hide
          kohsuke Kohsuke Kawaguchi added a comment -

          Fixed in 1.89. Thanks for the patch.

          Show
          kohsuke Kohsuke Kawaguchi added a comment - Fixed in 1.89. Thanks for the patch.

            People

            Assignee:
            Unassigned Unassigned
            Reporter:
            pfoorion pfoorion
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: