Description: ------------ The alone dot in a line is not replaced with two dots when it is on the beginning of the body. $body = ".\ntext"; $headers = "Subject: test"; $SMTP->data($body, $headers);

Another possibility of wrong dot handling is when body is a file handle to data stream with lonely dots in some lines, because it is parsed using fgets(). Consider text: "aaaaa . bbbbb ." I think any of dots here will be dot-stuffed. I think quotedata() should contain additional code, e.g.: if ($data == '.' || substr($data, 0, 3) == ".\r\n") $data = '.' . $data;

It should be "I think none of dots here will be dot-stuffed." above.

Thanks for hte report alec, for those of us not super familar with the package; what's the expected vs actual output of your test case? Expected: .. text Actual: . text ?

Exactly. In SMTP protocol <CR><LF>.<CR><LF> sequence is used as end of data mark. So, when it appears in the body it must be replaced with <CR><LF>..<CR><LF>.

This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.

The fix is wrong. See test case. "xxx.\n.xxx" => "xxx.\r\n..xxx", There's no line feed after the second dot. No need to dot-stuff it.

I don't think you're correct here. The SMTP standards says that we must dot-stuff whenever a line starts with a dot. From RFC-2821, Section 4.5.2 Transparency: - Before sending a line of mail text, the SMTP client checks the first character of the line. If it is a period, one additional period is inserted at the beginning of the line.