The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.From a maintenance perspective, it would be useful if there was a way of storing all of the emails being sent by the module, so that they could be monitored and problems could be debugged. Rather than writing custom code for this, just leverage the functionality in the Maillog module.
| Comment | File | Size | Author |
|---|---|---|---|
| #27 | smtp-n2653394-27.patch | 811 bytes | damienmckenna |
| #21 | smtp-n2653394-21.patch | 9.63 KB | damienmckenna |
Comments
Comment #2
damienmckennaComment #3
damienmckennaLets just use Maillog, it'll be easier.
Comment #4
damienmckennaThis adds optional integration with the maillog module.
Comment #6
damienmckennaThe tests are failing because of #2645590: Ensure that simpletest job doesn't "fail" testing if no tests are present .
Comment #7
damienmckennaThis version works much better.
Comment #9
damienmckennaComment #10
damienmckennaThis version trims the subject to 255 chars, the same as #2245761: E-mail subjects longer than 255 characters lead to a fatal error.
Comment #12
damienmckennaComment #13
damienmckennaThe last patch didn't create the $record->body field correctly.
Comment #15
damienmckennaComment #18
damienmckennaComment #20
damienmckennaRerolled.
Comment #21
damienmckennaThis adds tests to confirm that the email process stores the message in maillog. FYI #2653398: Add option to not actually deliver emails might be useful too, to avoid sending emails ;-)
Comment #22
damienmckennaComment #23
jordanpagewhite commentedI can confirm that this is working, both the devel functionality and the maillog database log. I am having an issue getting the MailLog module to properly set its menu items. I have posted the issue, on a pre-existing issue, in the MailLog issue queue here: https://www.drupal.org/node/2184669#comment-10805166
Comment #24
wundo commentedComment #26
wundo commentedComment #27
damienmckenna@wundo: Thanks, but you missed two files ;-)
Comment #28
wundo commentedSorry, the files were on my local, just fixed it.
Comment #30
wundo commentedComment #32
wundo commented