There is a string formatting error on date string set on line 350 in WireMailSmtpAdapter.php

$this->emailMessage->SetHeader("Date", gmdate("D, j M Y H:i:s \G\M\T P"));

Systems strictly observing RFC2822, will not accept the addition of the (Difference to Greenwich time (GMT) with colon between hours and minutes )added to the end date string created.

This causes issues with systems like Amazon AWS SES, Rackspace Mailgun, among others.

Line 350 should be changed to.

$this->emailMessage->SetHeader("Date", gmdate("D, j M Y H:i:s \G\M\T"));

https://www.ietf.org/rfc/rfc2822.txt

Credit goes to Rackspace tech support for isolating this.

feature request/idea. A bit of a head scratch. I am experiencing issues with clients changing their email account passwords and as a result breaking the module sending.

One thing is to log the sending/connection errors but the messages are long gone too. So, how about an email queue implementation where all messages would queue up if they cannot be sent for some reason and then upon fixing the issue whatever it might be the admin could resend the queued emails again?

There is lots to do I guess but I thought I'd throw it out there.

Hi @horst-n I was wondering if at some point in the future (no rush) you'd consider adding a namespace ProcessWire; to this module's PHP/module files so that PW doesn't have to compile it? I understand you've got several 3rd party classes that are in the root namespace and may need to remain so, so any references to them would need to be changed to have a backslash in front of them, like this:

$this->emailMessage = new \smtp_message_class();

Maybe an edge case, but when my SMTP server does not require authentication, the module test fails.
This is actually a false positive, even though the module test fails, the module still sends email as expected.
The offending code is part of the smtp.php library:

if($success
&& strlen($mechanism)==0)
{ 
$this->error="it is not supported any of the authentication mechanisms required by the server";    
$success=0;
}

sender_signature is added as plaintext to the text body and as html to html body (https://github.com/horst-n/WireMailSmtp/blob/master/WireMailSmtpAdaptor.php#L277).

This is clearly a bug, and should be fixed by

a) making the user supply both html and plaintext signatures

OR

b) correctly escaping the (plaintext) sig when injecting it to html payload.

Maybe a) and b) could be combined so that one can supply (plaintext) sig, and optionally also an html one. If an html sig is not supplied, the plaintext one is converted to html by escaping.

Hi @horst-n,

Microsoft will discontinue the SMTP Basic Auth method in 2025.

You will then only be able to send via Office 365 / Exchange using OAuth. Do you have any ideas or plans on how this critical change can be mapped with wireMailSMTP? Possibly via the OAuth library from the original author Manuel Lemos?

Best regards,
David

I feel like this: https://github.com/horst-n/WireMailSmtp/blob/master/WireMailSmtp.module#L64 should define the default as false, rather than ''

Does that sound right, or am I missing something?

Hi, i send some email and add the sender with
$email_message->SetHeader("Sender", "$name <$email>");
But if the email is reject from the destination server the undelivered message go back to the user of the httpd process.
Can i set a real envelope sender?

As the title says, there's an issue with CC - no matter which way you do the CC's (array of email addresses, associative array of email addresses and names) only the last person in the CC and the person in the To ever gets the email.

I was testing this with 3 recipients and the first CC was skipped over for some reason.

Under PHP 8.x and with the latest module version 0.6.0 there is a deprecated message when sending.

Deprecated: Function strftime() is deprecated in /site/modules/WireMailSmtp/smtp_classes/email_message.php on line 675

It may help to simply replace the function with Ryan's replacement, but this is also only available since PW 3.0.197. See: https://processwire.com/api/ref/wire-date-time/strftime/

Hi @horst-n - just wondering if this is intentional or an oversight?

Hi there,

I've just installed this awesome plugin that I'm using on pretty much all of my sites. But when testing whether the SMTP connection works, I'm getting the following error:

"Fatal Error: Uncaught Error: Class "WireMailSmtp" not found in site/modules/WireMailSmtp/WireMailSmtpAdaptor.php:143"

I'm using PW's latest dev version.

…is it safe to just remove this part of the condition?

1× PHP Deprecated: Function get_magic_quotes_runtime() is deprecated in .../smtp_classes/email_message.php:907

if((GetType($size)=="integer"&& strlen($body)>$size) || (function_exists("get_magic_quotes_runtime") && get_magic_quotes_runtime())) $body=StripSlashes($body);