PHPMailer causes "headers sent" errors when given invalid addresses

Bug #823058 reported by Richard Mansfield
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Low
Richard Mansfield

Bug Description

When a bad address is passed to PHPMailer, it produces output. When output_buffering is off in php.ini, a subsequent mahara redirect() call produces an unnecessary big red "Something bad happened after headers were sent" message. We should turn exceptions on in phpmailer, and catch them, to stop phpmailer from outputting its error messages. Then we can keep the "headers sent" messages for when we really need them.

Changed in mahara:
status: New → Confirmed
importance: Undecided → Low
assignee: nobody → Richard Mansfield (richard-mansfield)
milestone: none → 1.5.0
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/570
Committed: http://gitorious.org/mahara/mahara/commit/18d6f25bc4a545e8e9aada299944bb24403bf37c
Submitter: Hugh Davenport (<email address hidden>)
Branch: master

commit 18d6f25bc4a545e8e9aada299944bb24403bf37c
Author: Richard Mansfield <email address hidden>
Date: Tue Aug 9 16:45:26 2011 +1200

    Enable exceptions in phpmailer (bug #823058)

    This stops phpmailer from unnecessarily sending output to the page.

    Change-Id: Ib7ed71de4b5400aeaee39fee91fad27f0a7addb1
    Signed-off-by: Richard Mansfield <email address hidden>

Changed in mahara:
status: Confirmed → Fix Committed
Melissa Draper (melissa)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.