You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by pm...@apache.org on 2019/09/30 21:15:24 UTC

[jmeter] 01/07: Fix security warning by Sonar

This is an automated email from the ASF dual-hosted git repository.

pmouawad pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/jmeter.git

commit dbcd39f61a5775d8e676a6504ba6014318354f05
Author: pmouawad <p....@ubik-ingenierie.com>
AuthorDate: Mon Sep 30 23:10:31 2019 +0200

    Fix security warning by Sonar
---
 .../src/main/java/org/apache/jmeter/reporters/MailerModel.java         | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/components/src/main/java/org/apache/jmeter/reporters/MailerModel.java b/src/components/src/main/java/org/apache/jmeter/reporters/MailerModel.java
index f2f3ffd..8d35e0c 100644
--- a/src/components/src/main/java/org/apache/jmeter/reporters/MailerModel.java
+++ b/src/components/src/main/java/org/apache/jmeter/reporters/MailerModel.java
@@ -110,6 +110,8 @@ public class MailerModel extends AbstractTestElement implements Serializable {
 
     private static final String DEFAULT_LOGIN_VALUE = ""; //$NON-NLS-1$
 
+    private static final String MAIL_SMTP_CHECK_SERVER_IDENTITY = "mail.smtp.ssl.checkserveridentity"; //$NON-NLS-1$
+
     /** The listener for changes. */
     private transient ChangeListener changeListener;
 
@@ -347,6 +349,7 @@ public class MailerModel extends AbstractTestElement implements Serializable {
                 case SSL:
                     props.put(MAIL_SMTP_SOCKETFACTORY_CLASS,
                             "javax.net.ssl.SSLSocketFactory");
+                    props.put(MAIL_SMTP_CHECK_SERVER_IDENTITY, true);
                     break;
                 case TLS:
                     props.put(MAIL_SMTP_STARTTLS,