[imp] still have broken mail (header & body)

[Jeff Tucker]

--On Monday, February 24, 2003 9:39 PM -0700 Michael M Slusarz 
<slusarz at bigworm.colorado.edu> wrote:

> Quoting Jeff Tucker <jeff at jltnet.com>:
>
>| I updated Friday. I'm not sure when you posted your fix.
>
> Either friday afternoon or saturday.  You should update again.
>
> michael
>

Maybe there are two problems. I just updated and the problem which I 
personally am seeing is still present.

When sending messages using qmail on the imp server and imp in sendmail 
mode (calling qmail's sendmail compatibility program), there are carriage 
returns in the header lines which imp wraps, when there should only be 
linefeeds.

I know this is hard to read, but it shows the problem exactly. I can 
capture the message exactly as it is put into the mail queue, before it is 
ever sent via SMTP. Here are the first headers, as viewed with hexdump -c.

The important thing to note is that only the line which is wrapped by 
wrapHeaders has these \r\n line endings. Everything else just has \n.

If I change the call to extendedWordwrap in wrapHeaders and get rid of the 
\r, everything looks OK. My recollection is that sendmail fixes input 
errors like this for you, correcting the line endings when email is 
injected. By default, qmail's version does not. This might be viewed as 
qmail being incompatible with sendmail, but it's easy enough for imp to 
inject the mail correctly so that it doesn't matter.

0000000   R   e   c   e   i   v   e   d   :       (   q   m   a   i   l
0000010       4   8   5   8       i   n   v   o   k   e   d       b   y
0000020       u   i   d       9   9   )   ;       2   5       F   e   b
0000030       2   0   0   3       0   4   :   5   8   :   1   7       -
0000040   0   0   0   0  \n   R   e   c   e   i   v   e   d   :       f
0000050   r   o   m       a   d   s   l   -   3   5   -   4   9   -   1
0000060   9   4   .   a   s   m   .   b   e   l   l   s   o   u   t   h
0000070   .   n   e   t  \r  \n  \t   (   a   d   s   l   -   3   5   -
0000080   4   9   -   1   9   4   .   a   s   m   .   b   e   l   l   s
0000090   o   u   t   h   .   n   e   t       [   6   7   .   3   5   .
00000a0   4   9   .   1   9   4   ]   )       b   y       b   e   t   a
00000b0   .   c   e   s   m   a   i   l   .   n   e   t  \r  \n  \t   (
00000c0   H   o   r   d   e   )       w   i   t   h       H   T   T   P
00000d0       f   o   r       <   j   e   f   f   t   @   s   p   a   m
00000e0   c   o   p   .   n   e   t   @   c   e   s   m   a   i   l   .
00000f0   n   e   t   >   ;       M   o   n   ,       2   4       F   e
0000100   b       2   0   0   3  \r  \n  \t   2   3   :   5   8   :   1
0000110   7       -   0   5   0   0  \n   M   e   s   s   a   g   e   -
0000120   I   D   :       <   1   0   4   6   1   4   9   0   9   7   .
0000130   a   8   c   a   a   8   6   2   7   3   7   5   c   @   b   e
0000140   t   a   .   c   e   s   m   a   i   l   .   n   e   t   >  \n
0000150   D   a   t   e   :       M   o   n   ,       2   4       F   e
0000160   b       2   0   0   3       2   3   :   5   8   :   1   7
0000170   -   0   5   0   0  \n   F   r   o   m   :       j   e   f   f
0000180   t   @   s   p   a   m   c   o   p   .   n   e   t  \n   T   o
0000190   :       j   e   f   f   t   @   j   l   t   n   e   t   .   c
00001a0   o   m  \n   S   u   b   j   e   c   t   :       t   e   s   t


-- 
Jeff Tucker
jeff at jltnet.com