Docs: add note on HELO rejections, and add requirment on good HELO in
authorJeremy Harris <jgh146exb@wizmail.org>
Tue, 12 Jan 2016 17:52:30 +0000 (17:52 +0000)
committerJeremy Harris <jgh146exb@wizmail.org>
Tue, 12 Jan 2016 17:52:30 +0000 (17:52 +0000)
the example configuration

doc/doc-docbook/spec.xfpt
src/src/configure.default

index 6b4b5f3..44623a5 100644 (file)
@@ -27548,6 +27548,12 @@ Note that a client may issue more than one EHLO or HELO command in an SMTP
 session, and indeed is required to issue a new EHLO or HELO after successfully
 setting up encryption following a STARTTLS command.
 
+.new
+Note also that a deny neither forces the client to go away nor means that
+mail will be refused on the connection.  Consider checking for
+&$sender_helo_name$& being defined in a MAIL or RCPT ACL to do that.
+.wen
+
 If the command is accepted by an &%accept%& verb that has a &%message%&
 modifier, the message may not contain more than one line (it will be truncated
 at the first newline and a panic logged if it does). Such a message cannot
index ec60700..ee94d2f 100644 (file)
@@ -436,6 +436,11 @@ acl_check_rcpt:
           control       = submission
           control       = dkim_disable_verify
 
+  # Insist that a HELO/EHLO was accepted.
+
+  require message      = nice hosts say HELO first
+          condition    = ${if def:sender_helo_name}
+
   # Insist that any other recipient address that we accept is either in one of
   # our local domains, or is in a domain for which we explicitly allow
   # relaying. Any other domain is rejected as being unacceptable for relaying.