Hi Lorenzo,

Seems okay to me.

Current solution is wrong anyway, since Right( , 2 )
will rarely be equal to hb_osNewLine() on systems with
1 char long EOLs.

Pls commit it.

Brgds,
Viktor

On 2009 Sep 17, at 16:44, Lorenzo Fiorini wrote:

In contrib/hbtip/sendmail.prg these lines:

     IF !( Right( cBody, 2 ) == hb_osNewLine() )
        cBody += hb_osNewLine()
     ENDIF

should be:

     IF !( Right( cBody, 2 ) == Chr( 13 ) + Chr( 10 ) )
        cBody += Chr( 13 ) + Chr( 10 )
     ENDIF

the standard requires Cr+Lf and servers like QMail reject messages
with ONLY Lf reporting:

451 See http://pobox.com/~djb/docs/smtplf.html

while others like Postfix accept them but report:

354 End data with <CR><LF>.<CR><LF>

Using Chr( 13 ) + Chr( 10 ) seems to fix the send with QMail but
Postfix still reports 354 ...

Any comments is welcome.

best regards,
Lorenzo
_______________________________________________
Harbour mailing list
[email protected]
http://lists.harbour-project.org/mailman/listinfo/harbour

_______________________________________________
Harbour mailing list
[email protected]
http://lists.harbour-project.org/mailman/listinfo/harbour

Reply via email to