Re: [mantisbt-dev] Suggestion: enable access to the mailer error message

[Victor B. <vb...@gm...>]

Hi Jeferson,

I agree to the two suggestions.

1. We should supporting logging of sending attempts and logging of
errors (if we don't so already).

2. Another enhancements would be to store some more info in the
mantis_mail_table like number of retries and last error.  Then have a
page where users with the appropriate access level can view all the
queues messages and their corresponding details.  There is no need to
view the contents of the messages, maybe just the subjects, the target
email addresses, etc.

I think we should report these as two different feature requests.
Patches are welcome.

Does this make sense?

On 9/23/07, Jeferson Oliveira <jef...@gm...> wrote:
> Hi,
>
> After have suffered a hard to understand send mail problem (as I said
> here
> http://sourceforge.net/mailarchive/message.php?msg_name=fd171h%2434t%241%40sea.gmane.org
> and here
> http://sourceforge.net/tracker/index.php?func=detail&aid=1800644&group_id=26031&atid=385709)
> I would like to suggest that the error message received by Mantis from
> phpMailer could be acessible to the Mantis administrators.
>
> For this, in email_queue_api.php, we should add a property error_msg to
> the class EmailData (consequently create a field in the
> mantis_bug_file_table) to store the returned error msg, and maybe create
> a method email_queue_edit that would update the current record in the
> mantis_bug_file_table setting the value of the error_msg field in case
> of error. And finally in email-api.php line 771 change the email_send
> function from:
>
>                if ( !$mail->Send() ) {
>                        $t_success = false;
>                } else
>
>
> To something like:
>
>                if ( !$mail->Send() ) {
>                        $t_success = false;
>                        $t_email_data->error_msg = $mail->ErrorInfo;;
>                        email_queue_edit( $t_email_data);
>                } else
>
>
> Another option I see instead of store the message in the database, would
> be call the log_event function in case of an error
> (log_event(LOG_EMAIL_RECIPIENT, ...) so if the LOG_EMAIL_RECIPIENT (and
> also LOG_EMAIL?) is included in the $g_log_level variable the error
> message would be printed in the $g_log_destination.
>
> I would like to hear comments about this suggestions, and after discuss
> it I can report an issue in the Mantis bugtracker.
>
>
> Regards,
>
> Jeferson Oliveira
> Brazil
>
>
> Sent by Mozilla Thunderbird
> http://br.mozdev.org/thunderbird
>
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Microsoft
> Defy all challenges. Microsoft(R) Visual Studio 2005.
> http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
> _______________________________________________
> mantisbt-dev mailing list
> man...@li...
> https://lists.sourceforge.net/lists/listinfo/mantisbt-dev
>