From 451ecde5c915d94fdb6c5c8e59b291868290bc14 Mon Sep 17 00:00:00 2001
From: "mguessan@free.fr" <gdeYUExU5NGUKyzpdORa>
Date: Tue, 3 Sep 2019 13:05:31 +0200
Subject: [PATCH] Send FQDN instead of simple hostname in SMTP HELO command

This closes #101 pull request at github/apache/ant repo
---
 CONTRIBUTORS                                               | 1 +
 WHATSNEW                                                   | 7 +++++++
 contributors.xml                                           | 4 ++++
 src/main/org/apache/tools/mail/MailMessage.java            | 2 +-
 src/tests/junit/org/apache/tools/mail/MailMessageTest.java | 2 +-
 5 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/CONTRIBUTORS b/CONTRIBUTORS
index c269a6702..4d7a8cf6a 100644
--- a/CONTRIBUTORS
+++ b/CONTRIBUTORS
@@ -292,6 +292,7 @@ Michael Newcomb
 Michael Nygard
 Michael Saunders
 Michael Seele
+Mickaël Guessant
 Miha
 Mike Davis
 Mike Roberts
diff --git a/WHATSNEW b/WHATSNEW
index fa6604b03..4f199b811 100644
--- a/WHATSNEW
+++ b/WHATSNEW
@@ -1,6 +1,13 @@
 Changes from Ant 1.10.7 TO Ant 1.10.8
 =====================================
 
+=======
+Other changes:
+--------------
+
+ * org.apache.tools.mail.MailMessage will now send a fully qualified
+   domain name in its HELO message.
+   Github Pull Request #101
 
 Changes from Ant 1.10.6 TO Ant 1.10.7
 =====================================
diff --git a/contributors.xml b/contributors.xml
index 44c77e000..e73660bc9 100644
--- a/contributors.xml
+++ b/contributors.xml
@@ -1212,6 +1212,10 @@
     <first>Michael</first>
     <last>Seele</last>
   </name>
+  <name>
+    <first>Mickaël</first>
+    <last>Guessant</last>
+  </name>
   <name>
     <last>Miha</last>
   </name>
diff --git a/src/main/org/apache/tools/mail/MailMessage.java b/src/main/org/apache/tools/mail/MailMessage.java
index bb16c48f7..48b155059 100644
--- a/src/main/org/apache/tools/mail/MailMessage.java
+++ b/src/main/org/apache/tools/mail/MailMessage.java
@@ -393,7 +393,7 @@ public class MailMessage {
     }
 
     void sendHelo() throws IOException {
-        String local = InetAddress.getLocalHost().getHostName();
+        String local = InetAddress.getLocalHost().getCanonicalHostName();
         int[] ok = {OK_HELO};
         send("HELO " + local, ok);
     }
diff --git a/src/tests/junit/org/apache/tools/mail/MailMessageTest.java b/src/tests/junit/org/apache/tools/mail/MailMessageTest.java
index 944e8897d..72ee1fa19 100644
--- a/src/tests/junit/org/apache/tools/mail/MailMessageTest.java
+++ b/src/tests/junit/org/apache/tools/mail/MailMessageTest.java
@@ -44,7 +44,7 @@ public class MailMessageTest {
     @Before
     public void setUp() {
         try {
-            local = InetAddress.getLocalHost().getHostName();
+            local = InetAddress.getLocalHost().getCanonicalHostName();
         } catch (java.net.UnknownHostException uhe) {
             // ignore
         }