[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH]: Encoding fix mailer.py

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Fri, 9 Nov 2012 09:48:49 -0500

On 11/09/2012 09:31 AM, C. Michael Pilato wrote:
> On 09/04/2012 12:58 PM, Igor Galić wrote:
>> Hey folks,
>>
>> I've patched up my local mailer.py[1] to correctly encode
>> "From: " Headers.
>>
>> The attached patch *works* but it's not pretty.
>>
>> Someone who actually knows Python might want to encapsulate
>> my changes in a function.
>>
>> n.b.: I am not subscribed to this ML, if you want notify me
>> of updates, please CC me.
>>
>> So long,
>>
>> i
>>
>> [1] http://svn.apache.org/repos/asf/subversion/trunk/tools/hook-scripts/mailer/mailer.py
>>
>
> Igor, I took a look at your patch. I was confused by one aspect: why split
> the from_addr into words an encode each one individually, rather than
> encoding the whole header value like we do for the Subject: header? I dug
> around in the related RFCs, and if I'm reading the RFCs correctly, doing
> per-word encoding is actually the right thing to do. Should we, then, do
> the same for the Subject: header?

Attaching my current modification of your original patch.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Enterprise Cloud Development

Received on 2012-11-09 15:49:25 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.