Bug 460820

Summary: msg_footer gets its trailing spaces trimmed
Product: [Fedora] Fedora Reporter: Jan Kratochvil <jan.kratochvil>
Component: mailmanAssignee: Daniel Novotny <dnovotny>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: low Docs Contact:
Priority: low    
Version: rawhide   
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-10-29 08:46:55 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
a patch none

Description Jan Kratochvil 2008-09-01 14:49:13 UTC 
Description of problem:
After upgrading F9 to F10rawhide my mailing list started to corrupt the standard message signature delimiters.

Version-Release number of selected component (if applicable):
(broken) mailman-2.1.11-3.fc10.x86_64 (produces X-Mailman-Version: 2.1.11)
(correct) mailman on my box was producing: X-Mailman-Version: 2.1.9

How reproducible:
Tried once.

Steps to Reproduce:
1. Create a mailing list.
2. Put into its msg_footer:
-- $
_____________________________________________________$
%(real_name)s e-mail konference pro$
[ please cut the trailing single `$' character ]
3. Send a mail.
  
Actual results:
--$
_____________________________________________________$
%(real_name)s e-mail konference pro$

Expected results:
-- $
_____________________________________________________$
%(real_name)s e-mail konference pro$

Additional info:
This is a standard message signature described in many Internet documents:
http://www.guckes.net/mail/sig.etiquette.html#sigdashes

Having filter_content=no.
No digest delivery was involved.
Comment 1 Daniel Novotny 2008-10-27 10:56:12 UTC 
the code, which simply changed "\r\n" to "\n" in the previous version of Mailman, now strips trailing spaces from the lines: I asked upstream, why they made this change.
Comment 2 Daniel Novotny 2008-10-29 07:56:06 UTC 
Created attachment 321764 [details]
a patch

they want to preserve format=flowed in the
Content-Type: header, that is why they made this change:

adjusted in the way the footer is not trimmed
Comment 3 Jan Kratochvil 2008-10-29 08:04:57 UTC 
Thanks, OK.
Comment 4 Daniel Novotny 2008-10-29 08:46:55 UTC 
mailman-2.1.11-4.fc10 will be in F10 updates...