E-Mail issues with Login/Register/Profile Module

[pout]

Hi,

I have a proplem with the registration confirmation mail from this module. When I try to register with a gmx.at e-mail, the multipart confirmation mail is not shown correctly. Using a hotmail.com address, the outlook.com interface is showing no bodytext.

I tried to use WireMailSmtp and installed and configured the module. But it seems the e-mails are still send by the original WireMail. Don't know why.

Here is the sourcecode of the e-mail shown by thunderbird:

From - Thu Jan 24 18:15:36 2019
X-Account-Key: account4
X-UIDL: 1Mau6r-1hJrQE0cCB-00cSPh
X-Mozilla-Status: 0001
X-Mozilla-Status2: 00000000
X-Mozilla-Keys:                                                                                 
Return-Path: <postmaster@myhost.com>
Received: from corvus.xxx.de ([ip]) by mx-ha.gmx.net
 (mxgmx015 [212.227.15.9]) with ESMTPS (Nemesis) id 1MWSVd-1ggHUV3pmd-00Xp8U
 for <newuser@gmx.at>; Thu, 24 Jan 2019 18:15:24 +0100
Received: (qmail 18646 invoked from network); 24 Jan 2019 17:15:24 -0000
Received: from localhost (HELO arcturus.xxx.de) (127.0.0.1)
  by corvus.xxx.de with SMTP; 24 Jan 2019 17:15:24 -0000
Received: (qmail 16911 invoked by uid 1434); 24 Jan 2019 17:15:23 -0000
Date: 24 Jan 2019 17:15:23 -0000
Message-ID: <20190124171523.16910.qmail@arcturus.xxx.de>
To: newuser@gmx.at
Subject: myhost.com Konto =?UTF-8?Q?best=C3=A4tigen?=
X-PHP-Originating-Script: 1434:WireMail.php
From: processwire@myhost.com
X-Mailer: ProcessWire/WireMail

MIME-Version: 1.0

Content-Type: multipart/alternative;

  boundary="==Multipart_Boundary_x7bfa7a89930f498e5a91c658c9ec36acx"
Envelope-To: <newuser@gmx.at>
X-GMX-Antispam: 0 (Mail was not recognized as spam); Detail=V3;
X-Spam-Flag: NO
X-UI-Filterresults: notjunk:1;V03:K0:V2CsnrJkBGk=:yDYbM+J/8VAtWgrOXSZr8mKdPv
 r3syZlfeM3hGJvpI5LA+iqLR4CCri5VMng0f0UPpAIfz6ofyMYd7dz1wTP3I/KIVmJKRJlHHs
 Z3466LYl7AvMA1rl9ROa+BaAsX5x2wjSJyC3ITmmuAAeehGsgWpbbw0GSYj1vX1x/UYwovB3c
 lf+vYEtJyaeZyGvoX83D88huZnndfqbB3/kjGcM1iOKina3EeDcRjGaG8BRoGwikN4xqjYXel
 8esBpl/phVAO/v/g3TZW4P3VNt1TDqgRr7oA34G4ihexENVVdYrzXVpW36OBn6lg2zaHzWZwT
 I88XqNasJl7dlrqLzuWD5W58IRp87J6DoUHj1P88Jh3yoddOQc52AaEk00RO6Hs6IBk5I2xL0
 1PyCicddL3WMNmBywKGqbL9GElE1EFee1rzUQFLE0CmTH38bsv79hEf4REKqfdB1uMF0U6nnO
 zO5lXRIcuudJswlOdxPb6f0O3yITOgPXPEE7LtbSn4HhbrW3yMCtRLpCT4uMP8SWhgzV41ADk
 XLbTaT0wG40jIf154alk6dwP4tzor2kdNslYMpRWsJcsL/ljJ6YYzzzkM9DEtU5WnMJJ++zyq
 gRFsfFKsvlOpJGSi5CD42v6AP3zSsr0AKDqNk88nfeqMGDZCYssCcmQJvptSq7UlookvzUHgn
 LfVTNrscgzj/ar0h0NcgJpVYQ62EUrUCuf9R/hS8DWXmPq2dgqK44gpfo+I3naTaoxxhHdoI5
 sTAKqNw08qv2l0225D3FkXNe1OhensMG2fe5VFvnJIDK+OXABgBnDAEpPK3t+xNCX21PKkWGs
 vIOX2Ex6uxJRjLQ+aE2Ul2pHnJ0ZHJ5QKwMojCYbaO7jaaqbTSEUhAMflvjKP7BTK+9jSKTdG
 Z3+dgXRP4fEJQnEH4u38pnYyCwHkh1eamVJNuKt38B6WtjI1KmkcHgLtJ5AP6CFTuYPRJY7AU
 aOu0k03T47fURqHk/qf35QCivvutKPq82EaKan4owEG9g0bZPJsR2v1pG78hcit8V/7OoqJeL
 EKWpF2cctPbjDKCaOUBvCYs/ozoInko5RMcAPBs8pZHrEdF/LGWT2WIvMnu/CvL/ZW7RVfne6
 TSSWkEbfsOYJk2YFIGsDyBpPFmAP/V+CgEMjMf6FyT2oiYyG0OGACfcy97SlAfS627UpQSgUQ
 MSf/wKlcrxqXaBNffNlp7DYACxG7oIxMFtsS1l9HDkgjjGX7ta6aaPje/mQMoFTWtj2jJG4yA
 nANwzNV3wxzanL05iRl95wPhJf7sZ4VnJkswUHcqeXInUZLcPfZ5VL8Dg7SguOIgKazOHi8Rp
 oMCRTpoZipYRC9ps8TJOV+msbrv5PN2hU9+uAobHMjKgmKTYx2+IOz6kLLnlAZUIV7SRiitQv
 ZHPPTsu9aG0HjhjqRtETUGE6owwtHlpZluzLdlc+LploSyXg7dwqtXlDQNm2WWMoFF/NQAv+N
 pAInxEyGCqh62scqlneTxt1ssRyw9vg+SFa0qWemBC0TNTRcGSoKWQlVrwSlWQzcTTilKPYyq
 fR3l6MYM52QPZQsYHtEWaKxEoiyyIGSetz7pFP4X8kFsOcf8eBGmk5+ifyLDmHBwXuspZeIYR
 1nXo7Qzo6hrkNpiQGKkJp1O5IkhTDtYzLrOwtaosg9ttkiQl7HyUPylOTwWxHsxHYLsd3xZPH
 IQeqSld/jMgSiu0CKNwoCvLmx4pa9gsrPekZp+KVGR3jqEOjb32D86rs1r2+Tmjz5b21jEi6A
 9zt9/wjK0W1OsuzdNcjhBioS1eLtSJIN/flONkaBRlAuH0IwXdZ1puBmaq63zzw1WoqWsi2uc
 umGq2cDqcLnIYvqDhlYnSsDeD84jjVdZlD0/v7NcnheS321rxjQY6OXCa48xXUAIB4iXjMX9T
 a+jkAhCc5mzqxnbrHg/8fG0A3x51FSflJKrOLx7h/wEKniXte7Td8Ge55EIeNkUrRst7DkxaO
 exxeswvYBqAz7wvaEsozOR7ptncw63nBeB9X9g4hClZY6w83B4mEzh/Aw4CIAGD3+bNbEvGWV
 tiLKJQhfO7Mq7hD2Q9SLlnVxVrkz6PAtT+913goPq/

This is a multi-part message in MIME format.



--==Multipart_Boundary_x7bfa7a89930f498e5a91c658c9ec36acx

Content-Type: text/plain; charset="utf-8"

Content-Transfer-Encoding: quoted-printable



Bitte klicken Sie auf den folgenden Link, um ihr Konto auf myho=

st.com zu best=C3=A4tigen.=0A=0ABest=C3=A4tigungscode: uIqhuqIjchShXXX.w2ZtdH=

K9J0l8dg0W/lnXA5N5=0A=0Ahttps://myhost.com/mitglieder/?register_c=

onfirm=3DuIqhuqIjchShXXX.w2ZtdHK9J0l8dg0W%2FlnXA5N5



--==Multipart_Boundary_x7bfa7a89930f498e5a91c658c9ec36acx

Content-Type: text/html; charset="utf-8"

Content-Transfer-Encoding: quoted-printable



<p>Bitte klicken Sie auf den folgenden Link, um ihr Konto auf myh=

ost.com zu best=C3=A4tigen.</p>=0A=0A<p>Best=C3=A4tigungscode: uIqhuqIjchSh=
XXX.w2ZtdHK9J0l8dg0W/lnXA5N5</p><p><a href=3D'https://myhost.com/=

mitglieder/?register_confirm=3DuIqhuqIjchShXXX.w2ZtdHK9J0l8dg0W%2FlnXA5N5'>=

Zum Best=C3=A4tigen anklicken</a></p>



--==Multipart_Boundary_x7bfa7a89930f498e5a91c658c9ec36acx--

The bodyview in thunderbird shows everthing under the Line

MIME-Version: 1.0

Content-Type: multipart/alternative;

.....

 

Then I commented out the line 685 in LoginRegister.module to only send plain-Text. This worked with the gmx webinterface, but not in thunderbird (still shows source code in the bodypart) and outlook.com still showed an empty bodypart.

Any ideas form the e-mail specialists out there? Thanks

[adrian]

It's probably because that module uses:

new WireMail() 

https://github.com/ryancramerdesign/LoginRegister/blob/7e6395393d46f9fba3a6769cc2ce85d4e495b8a4/LoginRegister.module#L677

rather than:

wireMail() or $mail->new()

Docs here: https://processwire.com/api/ref/wire-mail/

That causes PW to use its core email sending, and prevents wireMailSMTP and other mailing modules from working. I think this is a critical bug in this module which needs attention. Can you please report here: https://github.com/ryancramerdesign/LoginRegister/issues

[pout]

Thanks!

Using

wireMail() 

instead of

new WireMail()

in line 677 made use of WireMailSmtp and this solved the other issues. I will post a bug report.

[BitPoet]

@pout: even the regular WireMail class shouldn't cause such problems, so it would be good if we could narrow the cause down. Could you check if the issue is still there if you add the following line after the one @adrian linked to:

	$mail->set("newline", "\n");

 

[pout]

@BitPoet: Yes you are right and yes, adding this line solved the issue also with the regular WireMail.

[adrian]

Just now, pout said:

@BitPoet: Yes you are right and yes, adding this line solved the issue also with the regular WireMail.

Good news that solved that issue, but the way wire mail is called in the module is still problematic and needs fixing.

[pout]

@adrian: I reportet this allready on github.

I will also report @BitPoets change.