[spambayes-bugs] [ spambayes-Bugs-797316 ] Extra CRLF to smtp server causes garbage error.

[SourceForge.net]

Bugs item #797316, was opened at 2003-08-30 02:29
Message generated for change (Comment added) made by anadelonbrin
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=498103&aid=797316&group_id=61702

Category: pop3proxy
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Dudley Fox (grieve)
Assigned to: Nobody/Anonymous (nobody)
Summary: Extra CRLF to smtp server causes garbage error.

Initial Comment:
In version 1.0a4 of smtpproxy.py on line 200 it reads
     self.serverSocket.push(cooked + '\r\n')
It should read
     self.serverSocket.push(cooked)

The first line sends an empty command to the smtp
server, and causes it to return an error. Different
smtp servers and mail clients handle it differently,
but Eudora will complain that when it sent the QUIT
command an error was returned. Agent will just fail
silently. In both cases the message is actually sent.

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

>Comment By: Tony Meyer (anadelonbrin)
Date: 2003-08-30 20:14

Message:
Logged In: YES 
user_id=552329

You are correct that there is a problem that Eudora complains 
about an error after the QUIT command.

However, if the '\r\n' is removed, then *no* commands work 
(at least here).  Certainly this is the way that the RFC says 
that SMTP is meant to work (specifically, that there is a CRLF 
after each command, including QUIT).

Can you confirm for me that removing the '\r\n' means that 
you can still use smtpproxy?  If I remove it here, it hangs 
forever on the first command sent (ELHO, in this case).

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

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=498103&aid=797316&group_id=61702