Issue #377 has been updated by Krzysztof Stryjek.

Yes, I've tried this. But then my SMTP server logs said:
[email protected] -> [email protected]
[email protected] -> [email protected]
[email protected] -> [email protected]
[email protected] -> [email protected]

So 4 mails were generated instead of one. Problem is that $mail_from is used as:

    /* Send the mail */
    if ($mail_from) {
        $result = mail($mail, $subject, $body, $header."From: 
$mail_from\r\n","-f$mail_from");
So you can see that $mail_from is used as an argument for local sendmail 
(according to mail() PHP manual).

I tried to code this as:

    /* Send the mail */
    if ($mail_from) {
        $header .= "From: ";
        if( $GLOBALS['messages']['resetfrom'] ) {
               $header .= "".mb_encode_mimeheader( 
$GLOBALS['messages']['resetfrom'] )." ";
        }
        $header .= "<".$mail_from."> \n";
        $result = mail($mail, $subject, $body, $header, "-f$mail_from");

(of course $GLOBALS['messages']['resetfrom'] should be replaced with function 
argument like $mail_from_txt).

What do you think about it?

----------------------------------------
Feature #377: Adding mail_from_txt to send_mail function - SSP
http://tools.lsc-project.org/issues/377

Author: Krzysztof Stryjek
Status: New
Priority: Normal
Assigned to: 
Category: Self Service Password
Target version: self-service-password-?


Hello,

I think it is good idea to add resetfrom message (like 'resetsubject') to 
send_mail function. So user will receive message like:
From: Password Manager <[email protected]>
Subject: $message['resetsubject']
...
...

I can see that this function is called in four files, so it will be very easy 
to add this feature. I will try add some patches in few days


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://tools.lsc-project.org/my/account
_______________________________________________
ltb-dev mailing list
[email protected]
http://lists.ltb-project.org/listinfo/ltb-dev

Reply via email to