httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Colm MacCarthaigh <c...@stdlib.net>
Subject Re: [PATCH] mod_smtpd_queue_smtp
Date Sun, 18 Sep 2005 09:29:37 GMT
On Sat, Sep 17, 2005 at 04:40:14PM -0700, Garrett Rooney wrote:
> +static const command_rec smtpd_queue_smtp_cmds[] = {
> +    AP_INIT_FLAG("SmtpQueueSmtp", enable_queue_smtp, NULL, RSRC_CONF,
> +                 "Enable queuing to another SMTP server.  Default: Off"),
> +
> +    AP_INIT_TAKE1("SmtpQueueSmtpHost", set_queue_smtp_host, NULL, RSRC_CONF,
> +                  "Set the hostname for the remote SMTP server."),
> +
> +    AP_INIT_TAKE1("SmtpQueueSmtpPort", set_queue_smtp_port, NULL, RSRC_CONF,
> +                  "Set the port for the remote SMTP server."),
> +
> +    { NULL }
> +};

These seem really unintuitive names; SMTP Queueing usually means
managing a spool and incremental backoff, ie the administration of a
mail queue. 

But as far as I can tell, this code is all about SMTP forwarding (not
even relaying per-se). Confuses me anyway :)

Might be worth using EHLO instead of HELO from day one, just for interop
testing. We'll likely need EHLO for TLS and so on anyway.

-- 
Colm MacCárthaigh                        Public Key: colm+pgp@stdlib.net

Mime
View raw message