Giuliano wrote:
>
>On 17 Mar 2006, at 06:37, Mark Sapiro wrote:
>
>> I agree that it's a bug, but I think the primary bug is in an MUA that
>> is generating a reply to mail
>>
>> From: [EMAIL PROTECTED]
>>
>> and addresses it
>>
>> To: "[EMAIL PROTECTED]" <[EMAIL PROTECTED]>
>>
>
>Not exactly, it does only add the local_part and without quotes.


But does it ever do it in the case we are concerned with? Namely where
the message is

From: [EMAIL PROTECTED]

Note that the original regexp works as long as the MUA added stuff
doesn't contain '+'.


>I have tested your regexp for a "regular" reply and it works. Of  
>course it does not work with that buggy format (wherever one consider  
>the bug to be...) and the fixed version is below.


I'm not sure what "your regexp" means here. Is it the one that is
currently in Defaults.py(.in), or my suggested change?


>> However, I have no objection to Mailman "working around" this MUA bug
>> :-) by changing
>>
>> VERP_CONFIRM_REGEXP =r'^(?P<addr>[^+]+?)\+(?P<cookie>[EMAIL PROTECTED])@.*$'
>>
>> in Defaults.py.in
>>
>> I ask though, does anyone have a better suggestion for a replacement
>> than
>>
>> VERP_CONFIRM_REGEXP = \
>>  r'^(\s*"[^"]*")?(?P<addr>[^+]+?)\+(?P<cookie>[EMAIL PROTECTED])@.*$'
>>
>> or is that OK?
>
>it is too specific as I said. This works instead:


Yes, it requires quotes, but as I ask above. Does any MUA that adds
this not put quotes around it when it contains '+'?


>VERP_CONFIRM_REGEXP = r'^(\s*.*)?(?P<addr>[^+]+?)\+(?P<cookie>[EMAIL 
>PROTECTED]) 
>@.*$'
>
>This was checked against many formats and  probably the simpler and  
>less reduntant:
>
>VERP_CONFIRM_REGEXP = r'^(.*)?(?P<addr>[^+]+?)\+(?P<cookie>[EMAIL 
>PROTECTED])@.*$'
>
>would work too. Or even:
>
>VERP_CONFIRM_REGEXP = r'(.*)?(?P<addr>[^+]+?)\+(?P<cookie>[EMAIL 
>PROTECTED])@.*$'
>
>as I do not see the point of checking for the line beginning.


Well, I don't think the 'addr' field is actually used, but I think
these suggestions eat up all but the last character of it. How about

VERP_CONFIRM_REGEXP = r'(.*<)?(?P<addr>[^+]+?)\+(?P<cookie>[EMAIL 
PROTECTED])@.*$'

-- 
Mark Sapiro <[EMAIL PROTECTED]>       The highway is for gamblers,
San Francisco Bay Area, California    better use your sense - B. Dylan

_______________________________________________
Mailman-Developers mailing list
[email protected]
http://mail.python.org/mailman/listinfo/mailman-developers
Mailman FAQ: http://www.python.org/cgi-bin/faqw-mm.py
Searchable Archives: 
http://www.mail-archive.com/mailman-developers%40python.org/
Unsubscribe: 
http://mail.python.org/mailman/options/mailman-developers/archive%40jab.org

Security Policy: 
http://www.python.org/cgi-bin/faqw-mm.py?req=show&file=faq01.027.htp

Reply via email to