On 2004-04-14 19:41:10 -0400, Keith C. Ivey wrote: > Joe Stewart <[EMAIL PROTECTED]> wrote: > > > I had to change line 11 of require_resolvable_fromhost from: > > > > $sender->format ne "<>" > > > > to: > > > > $sender->format ne "\"<>\"" > > > > because of the way Mail::Address adds quotation marks to an > > address. > > There was some discussion of that in November in the thread > "PATCH: <> rejected by require_resolvable_fromhost". You can > find messages here: > > http://www.nntp.perl.org/group/perl.qpsmtpd;max=713 > > I don't know whether anything was really resolved. > > It seems to me that Mail::Address objects aren't what we should > be using for addresses,
I agree with that. Mail::Address is very forgiving, while the syntax in
SMTP is quite strict.
> since the addresses in the MAIL and RCPT parameters are not RFC 822
> addresses (they don't have angle brackets and name and comments and
> such).
Actually, they do have angle brackets, and the brackets are required,
while they are optional in RFC 822.
> Maybe they should be a Qpsmtpd::Address object,
Yup.
> and there could be a local() method to indicate whether the address is
> local.
Hmm. How do you determine that? Do we need a new hook for that?
hp
--
_ | Peter J. Holzer | I think we need two definitions:
|_|_) | Sysadmin WSR | 1) The problem the *users* want us to solve
| | | [EMAIL PROTECTED] | 2) The problem our solution addresses.
__/ | http://www.hjp.at/ | -- Phillip Hallam-Baker on spam
pgp00000.pgp
Description: PGP signature
