On Nov 25, 2003, at 12:54 PM, Matt Sergeant wrote:
Ugh. It looks like the way Mail::Address::format() uses it, new('','') is
most correct. New('<>') is definitely incorrect, though new('') will work.
Can you please come up with a test that shows that new('<>') is incorrect? I'm not going to just blindly apply this patch without a test to show exactly what you're trying to achieve (that's not to say that Ask won't apply the patch of course).
I'd like to see a test as well. Speaking of which, we really should start adding tests to qpsmtpd. In particular a framework for testing plugins would be very very neat.
I have code that checks for the <> envelope sender, and it works fine, so I'm wondering who's misunderstanding what here.
yeah, without testing then I'm slightly confused as well. I use qpsmtpd from CVS as of a month or two ago on my servers and I haven't had any problems (at least none that could be related to this).
-- http://www.askbjoernhansen.com/
