[MDL-37152] SMTP mailer is set, even though configured not to use SMTP - Moodle Tracker

XML

Word

Printable

Details

Type: Bug
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: 2.2.6, 2.3.3, 2.4, 2.5
Fix Version/s: 2.2.7, 2.3.4, 2.4.1
Component/s: Administration, Messages
Labels:
- triaged

Affected Branches:
MOODLE_22_STABLE, MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE
Fixed Branches:
MOODLE_22_STABLE, MOODLE_23_STABLE, MOODLE_24_STABLE
Pull from Repository:
git://github.com/skodak/moodle.git
Pull Main Branch:
w51_~~MDL-37152~~_m25_smtp
Pull Main Diff URL:
https://github.com/skodak/moodle/compare/master...w51_MDL-37152_m25_smtp
Testing Instructions:

Hide

0/ configure moodle to use SMTP server for mail sending and smtpmaxbulk > 1
1/ trigger some action that mails multiple emails via SMTP (such as forum mailing to multiple users). Verify mails are sent.
2/ repeat with other non-smtp mail type (hint: set smtp hosts to 'qmail' or leave empty for sendmail). Verify no "SMTP Error: Could not connect to SMTP host" is logged / shown and mails are sent.

Show
0/ configure moodle to use SMTP server for mail sending and smtpmaxbulk > 1 1/ trigger some action that mails multiple emails via SMTP (such as forum mailing to multiple users). Verify mails are sent. 2/ repeat with other non-smtp mail type (hint: set smtp hosts to 'qmail' or leave empty for sendmail). Verify no "SMTP Error: Could not connect to SMTP host" is logged / shown and mails are sent.

Description

We are experiencing the same issue as described in this bug report: http://tracker.moodle.org/browse/MDL-29494

Users are getting this error message: "SMTP Error: Could not connect to SMTP host", although our configuration uses default PHP mail function (smtphosts is empty).

Looking at the source I found that $mail->IsSMTP() is being called in email_to_user function after a message is successfully sent (lib/moodlelib.php line 5469 in MOODLE_23_STABLE, 5505 in MOODLE_24_STABLE).

A quick workaround is to set smtpmaxbulk to 1 - so that the same incorrectly reconfigured mailer is not reused for next messages.

Attachments

Activity

People

Assignee:: Petr Skoda

Reporter:: Aurelijus Bruzas

Integrator:: Eloy Lafuente (stronk7)

Tester:: Rossiani Wijaya

Participants:: Aurelijus Bruzas, Eloy Lafuente (stronk7), Petr Skoda, Rajesh Taneja, Rossiani Wijaya

Votes:: 0 Vote for this issue

Watchers:: 6 Start watching this issue

Dates

Created:: 14/Dec/12 2:43 AM

Updated:: 12/Jan/13 1:17 AM

Resolved:: 12/Jan/13 1:17 AM

Fix Release Date:: 14/Jan/13