Bug 1222046 Comment 44 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Ok, I have now duplicated what Philippp sees. I modified his attachment 9111214 [details] with a From field and set up a filter to forward the message. It fails with a timeout and leaves behind the /tmp/nsemail.eml file with no CRLF end of line marker after the html email body. 
This happens only when setting account to compose as html. If the account is set to not compose as html, the message is forwarded as is, plain text and works OK with no timeout (and no /tmp/nsemail.eml is left behind) so I assume the message body ends with CRLF.

Not sure if this is similar to what other reporters are seeing, but I suspect it is and I think an easy solution is for tb always send "CRLF.CRLF" as the smtp termination sequence and not just ".CRLF" like it does now in this function: https://searchfox.org/comm-central/rev/7bf30c9070215a6fcb51fe51fc89bfd5e8c7e6fb/mailnews/base/util/nsMsgProtocol.cpp#1216

Note again that the "try" build linked above in comment 35 won't fix the problem if what Philippp observed applies to the other commenters.
Ok, I have now duplicated what Philippp sees. I modified his attachment 9111214 [details] with a From field and set up a filter to forward the message. It fails with a timeout and leaves behind the /tmp/nsemail.eml file with no CRLF end of line marker after the html email body. 
This happens only when setting account to compose as html. If the account is set to not compose as html, the message is forwarded as is, plain text and works OK with no timeout (and no /tmp/nsemail.eml is left behind) so I assume the message body ends with CRLF.

Not sure if this is similar to what other reporters are seeing, but I suspect it is and I think an easy solution is for tb always send "CRLF.CRLF" as the smtp termination sequence and not just ".CRLF" like it does now in this function: https://searchfox.org/comm-central/rev/7bf30c9070215a6fcb51fe51fc89bfd5e8c7e6fb/mailnews/base/util/nsMsgProtocol.cpp#1216
Edit: Found the root cause of the problem and fixing it there (see comment 49).

Note again that the "try" build linked above in comment 35 won't fix the problem if what Philippp observed applies to the other commenters.

Back to Bug 1222046 Comment 44