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