Matt Sergeant <[EMAIL PROTECTED]> wrote:

> Would it 
> be better to have a method on the Connection (or some other) class that 
> does a (member || ENV || config) bit of code and includes a setter for 
> the member variable?

I see now that the only environment variables for qmail-smtpd 
are RELAYCLIENT and DATABYTES, so the method wouldn't have to 
be a general one.  There could be two separate methods that 
might be different, if there's a reason for them to be.

Another point I noticed while looking at the qmail-smtpd man 
page is that the value of $ENV{RELAYCLIENT} is supposed to be 
appended to the recipient.  Presumably that means SMTP.pm needs

   $rcpt .= $ENV{RELAYCLIENT} if defined $ENV{RELAYCLIENT};

in rcpt() before adding the recipient.

-- 
Keith C. Ivey <[EMAIL PROTECTED]>
Washington, DC

Reply via email to