Re: Approved line bug

Michael Yount <[email protected]> Wed, 31 Oct 2007 12:30:52 -0700
Newsgroups gmane.mail.majordomo.majordomo2.devel
Message-ID <[email protected]>
This misformatted message is due to a bug in the processing of an 
Approved line in the body of a single-part message.  If the Approved 
line is followed by a non-blank line, the code assumes that the 
following lines constitute the headers and body of the message that 
should be posted.  This resulted in Mj2 sending only the body of John's 
message.

I have applied a patch which checks that the succeeding line is a header 
before discarding the original headers.  The Approved line will still be 
stripped before the message is posted.

Michael


mj2-dev-owner wrote:
> Did that, the message looks OK when presented for approval, then has lots 
> of blank lines when delivered.
>
>   
>>> Added clue: some copies have more blank lines than others, which makes no 
>>> sense since I use qmail and they should all be sent at once.
>>>       
>> Are the blank lines \n or \r\n or something else?
>>     
>
> I believe they're \n which qmail turns to \r\n.
>
> R's,
> John
>
>