Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Hello,
On our staging server, we use Devel module's "SMTP Library:" option, which lets you choose "Default" or "Log Only." It's set to only Log on the staging site so our users don't receive erroneous or duplicate emails. This does not seem to stop Mimemail from going out - at least Mimemail's rules implementation. Is this by design or an accident? Mimemail is the only module I know of that ignores this directive.
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | mimemail-1205682-8.patch | 1.43 KB | sgabe |
| #4 | mimemail-disable-option-1205682-4.patch | 708 bytes | smithmilner |
| #3 | mimemail-disable-option-1205682-3.patch | 708 bytes | smithmilner |
Comments
Comment #1
sgabe CreditAttribution: sgabe commentedThis is by design.
Mime Mail can send a message by itself after just preparing it with drupal_mail(). The Devel module defines drupal_mail_wrapper() when you set "Log Only", but that won't catch messages sent by Mime Mail, just the system messages which are sent with drupal_mail() by default and only processed by Mime Mail if it's set to be used for all messages.
Comment #2
Offlein CreditAttribution: Offlein commentedsgabe, sorry, I wish I'd replied to this long ago. I don't know why I didn't. Sorry to rudely reopen this ticket after so long.
My main point is that I'm curious if there's an existing way to disable Mimemail unilaterally somehow, preferably by a Drupal variable. I need to have a staging server that is an exact duplicate of our production environment except for a few features. For instance, it can't be sending automated emails to people, and it can't be automatically charging users' credit cards (something we do for recurring billing).
In the past, as I said, I have simply set the staging environment's settings.php up with a hardcoded variable setting the Drupal "SMTP library" option to use Devel's "Log Only", but as you confirmed, that won't work with Mimemail.
So, is there a variable I could hardcode in my staging settings.php to effectively disable any Mimemail emailings? I notice there is a "mimemail_engine" variable - perhaps I could change that?
Comment #3
smithmilner CreditAttribution: smithmilner commentedmarked http://drupal.org/node/1637766#comment-6134064 as a duplicate of this ticket.
There is no disable option for mimemail that I can see but it is certainly needed. Let's see if we can add a safeguard here. I've included a small patch that allows developers to create a variable to disable mimemail or disable mimemail if the smtp_library is set to devel.
Comment #4
smithmilner CreditAttribution: smithmilner commentedMy mistake, previous patch still sends messages if either the disable variable or the smtp_library are approved. If should stop messages if either is denied.
Comment #5
sgabe CreditAttribution: sgabe commentedYour patch will send the message when mimemail() is called not to send but to prepare with the
$send = FALSEparameter and none of the conditions met.Is a new variable for this really necessary? Devel is a very popular module and it has this functionality, Mime Mail should just simply respect it.
Comment #6
smithmilner CreditAttribution: smithmilner commentedNo a new variable isn't necessary, we can exclude it. Perhaps we should look at a way of logging the emails to watchdog when devel is configured as the smtp_library. That's what it does with other emails and people might get confused that Mime Mail might not be working.
Comment #7
aidanlis CreditAttribution: aidanlis commentedI'd really like this to be included! Logging would be very helpful.
Comment #8
sgabe CreditAttribution: sgabe commentedI am attaching a patch with logging, though I am not sure about the way. Since we are talking about MIME encoded messages with possible attachments and embedded images it does not seem to be a good idea to log the message body.
Comment #9
TR CreditAttribution: TR commentedNew features should go into the current version of Mime Mail. If this feature is important to you please contribute a patch, a test, or provide support in other ways.
Comment #10
TR CreditAttribution: TR commentedThe use case described in #2 can be handled in many ways. See https://www.drupal.org/docs/develop/local-server-setup/managing-mail-han...
Mime Mail should not be implementing features that are better handled by other contributed modules. Mime Mail only does formatting of emails. Delivery (or non-delivery, or logging, etc.) of emails should be be left to other modules as described above.