Add a comment in smtp_greeting() warning that the fqdn

*must* appear immediately after the 220 status code, and this
should never be changed.

diff --git a/citadel/serv_smtp.c b/citadel/serv_smtp.c
index 39128f4667dea4a473b84219f67a7e0a6993fdda..d4ff45a83a73c053d85c0a9ba30d880ef86da9b3 100644 (file)
--- a/citadel/serv_smtp.c
+++ b/citadel/serv_smtp.c
@@ -155,6 +155,9 @@ void smtp_greeting(int is_msa)
                return;
        }
 
+       /* Note: the FQDN *must* appear as the first thing after the 220 code.
+        * Some clients (including citmail.c) depend on it being there.
+        */
        cprintf("220 %s ESMTP Citadel server ready.\r\n", config.c_fqdn);
 }