[Alerting] for email action, set tls.rejectUnauthorized: false when secure: false (#62380)

pmuellr · pmuellr · commit d808aeef632c · 2020-04-07T08:31:50.000-04:00
resolves #62372 See the referenced issue for background. Eventually we will probably have to have a separate setting for `tls.rejectUnauthorized`, not base it on the value of the `secure` config property. But this will likely be useful for a number of smtp servers used by customers.
diff --git a/x-pack/plugins/actions/server/builtin_action_types/lib/send_email.ts b/x-pack/plugins/actions/server/builtin_action_types/lib/send_email.ts
@@ -65,6 +65,11 @@ export async function sendEmail(logger: Logger, options: SendEmailOptions): Prom
     transportConfig.host = host;
     transportConfig.port = port;
     transportConfig.secure = !!secure;
+    if (!transportConfig.secure) {
+      transportConfig.tls = {
+        rejectUnauthorized: false,
+      };
+    }
   }
 
   const nodemailerTransport = nodemailer.createTransport(transportConfig);
