[ mailman-Patches-670167 ] fix i18n attachment description in archive

[SourceForge.net]

Patches item #670167, was opened at 2003-01-18 05:58
Message generated for change (Comment added) made by tkikuchi
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=300103&aid=670167&group_id=103

Category: Pipermail
Group: Mailman 2.1
>Status: Closed
Resolution: Accepted
Priority: 8
Submitted By: Tokio Kikuchi (tkikuchi)
Assigned to: Nobody/Anonymous (nobody)
Summary: fix i18n attachment description in archive 

Initial Comment:
Bug-ID 669081 (Major attachment handling bug!) is
caused inaccurate handling of charset in Scrubber.py.
This is caused   by misuse of email functions (or by
bug in email package).
In Scrubber.py, attempt is made to get message charset
by part.get_charset() but it returns only None, as far
as I experimented. In this patch, message charset is
extracted from regular expression pattern in
content-type header.
This also set the charset of internally crafted message
part.
 

----------------------------------------------------------------------

>Comment By: Tokio Kikuchi (tkikuchi)
Date: 2003-12-25 12:40

Message:
Logged In: YES 
user_id=67709

I have opened a new tracker for collection of i18n patch
#865661. So, closing this patch.


----------------------------------------------------------------------

Comment By: Guillaume Rousse (guillomovitch)
Date: 2003-11-14 09:36

Message:
Logged In: YES 
user_id=144568

What is the status of this patch for mailman 2.1.3 ? 

----------------------------------------------------------------------

Comment By: Hugh Caley (hcaley)
Date: 2003-05-30 17:23

Message:
Logged In: YES 
user_id=17460

I patched version 2.1.2 of Mail man with patch
Scrubber.py.patch.030224.txt, using "patch -p0
<Scrubber.py.patch.030224.txt".  The patch applied cleanly,
other than removing a couple of CR's at the end of the file.
 However, it doesn't seem to make a difference; messages
sent from Outlook XP with an attachment still lose their
message text.  Did I do this correctly?

Hugh_Caley at affymetrix.com

----------------------------------------------------------------------

Comment By: Tokio Kikuchi (tkikuchi)
Date: 2003-04-22 13:00

Message:
Logged In: YES 
user_id=67709

Recently, I had a chance to get personal correspondence to
refer this patch. To quote his comment, "(it) works like a
charm!"
So, I am incrementing the priority. :^)


----------------------------------------------------------------------

Comment By: Tokio Kikuchi (tkikuchi)
Date: 2003-02-24 04:34

Message:
Logged In: YES 
user_id=67709

Uploading revised patch to fix more error.
Sorry for the people downloaded earlier.


----------------------------------------------------------------------

Comment By: Tokio Kikuchi (tkikuchi)
Date: 2003-02-24 01:39

Message:
Logged In: YES 
user_id=67709

update my patch to fix error reported by sumeet

  File "/list/Mailman/Handlers/Scrubber.py", line 320, in
process
    if not t.endswith('\n'):
AttributeError: 'NoneType' object has no attribute 'endswith'


----------------------------------------------------------------------

Comment By: Tokio Kikuchi (tkikuchi)
Date: 2003-02-22 07:05

Message:
Logged In: YES 
user_id=67709

This patch for Scrubber.py now include Bug-ID #691039.
save_attachment() now returns (url, size), because in
previous code, get_paylaod() is called before calling
save_attachment() only to get the size. save_attachment()
calls get_payload() inside.
We might be able to discard the attachment if
get_payload(decode=1) fail. (TBD: return discard message and
size 0 ?)


----------------------------------------------------------------------

Comment By: Tokio Kikuchi (tkikuchi)
Date: 2003-02-20 01:31

Message:
Logged In: YES 
user_id=67709

Increment priority because this is now a bug fix.


----------------------------------------------------------------------

Comment By: Tokio Kikuchi (tkikuchi)
Date: 2003-02-04 06:08

Message:
Logged In: YES 
user_id=67709

I made a new patch to fix embedded multipart messages are
stripped bug. this patch is for CVS.
see
http://mail.python.org/pipermail/mailman-users/2003-February/026071.html


----------------------------------------------------------------------

Comment By: Barry A. Warsaw (bwarsaw)
Date: 2003-01-20 05:13

Message:
Logged In: YES 
user_id=12800

Reopening so this issue doesn't get lost.

----------------------------------------------------------------------

Comment By: Tokio Kikuchi (tkikuchi)
Date: 2003-01-20 04:03

Message:
Logged In: YES 
user_id=67709

We, Japanese, need additional process of stringify and
regenerate  message instance..
Internal message is in 'euc-jp' but return value of
get_content_charset() becomes 'iso-2022-jp' even though the
get_payload() returns euc-jp text.
The message and part body is converted to iso-2022-jp when
it is output to SMTP (or whatever accepts in string format).


----------------------------------------------------------------------

Comment By: Barry A. Warsaw (bwarsaw)
Date: 2003-01-20 02:18

Message:
Logged In: YES 
user_id=12800

Accepted, with changes.

----------------------------------------------------------------------

Comment By: Tokio Kikuchi (tkikuchi)
Date: 2003-01-19 00:52

Message:
Logged In: YES 
user_id=67709

Update of patch.
Please backout the first one if you have already applied.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=300103&aid=670167&group_id=103