Hi Jason,

On Wed, May 14, 2014 at 11:16:59PM +0059, Jason McIntyre wrote:
> it's still missing some <>. also i think we can wholly simplify the
> auth/auth-optional line.

Heh. Thought I found all, thanks for fixing the rest.

> -.Op Ic auth | auth-optional | auth Ar authtable | Ic auth-optional Ar 
> authtable
> +.Op Ic auth | auth-optional Aq Ar authtable

the man page sates that:

 Both auth and auth-optional accept an optional table as a
 parameter.  When provided, credentials are looked up in this
 table.

but reading the new smtpd.conf(5) gives the impression that authtable is
mandatory. I haven't checked the code but the following passes a check
with smtpd -n

  listen on iwn0 tls auth
  listen on em0 tls auth-optional

so it seems to be a valid syntax.

Index: usr.sbin/smtpd/smtpd.conf.5
===================================================================
RCS file: /cvs/src/usr.sbin/smtpd/smtpd.conf.5,v
retrieving revision 1.118
diff -u -r1.118 smtpd.conf.5
--- usr.sbin/smtpd/smtpd.conf.5 15 May 2014 19:36:45 -0000      1.118
+++ usr.sbin/smtpd/smtpd.conf.5 15 May 2014 20:40:52 -0000
@@ -594,7 +594,9 @@
 .Op Ic port Ar port
 .Op Ic tls | tls-require | tls-require verify | smtps | secure
 .Op Ic pki Ar pkiname
-.Op Ic auth | auth-optional Aq Ar authtable
+.Oo Ic auth | auth-optional 
+.Op Aq Ar authtable
+.Oc
 .Op Ic tag Ar tag
 .Op Ic hostname Ar hostname
 .Op Ic hostnames Aq Ar names

Reply via email to