X-Git-Url: http://git.vanrenterghem.biz/git.ikiwiki.info.git/blobdiff_plain/2f8123012c9a6ad077b856d7f801346a4d9800fe..2c299c7994a4ab4decde1734f52c59a7103e85d2:/doc/todo/passwordauth:_sendmail_interface.mdwn

diff --git a/doc/todo/passwordauth:_sendmail_interface.mdwn b/doc/todo/passwordauth:_sendmail_interface.mdwn
index 0ae413413..556240964 100644
--- a/doc/todo/passwordauth:_sendmail_interface.mdwn
+++ b/doc/todo/passwordauth:_sendmail_interface.mdwn
@@ -1,4 +1,4 @@
-[[tag wishlist]]
+[[!tag wishlist plugins/passwordauth]]
 
 For sending out password reminder emails, the [[plugins/passwordauth]] plugin currently uses
 the *[Mail::Sendmail](http://search.cpan.org/perldoc?Mail::Sendmail)* module.
@@ -23,3 +23,39 @@ I'd offer to work on a patch for the [[plugins/passwordauth]] plugin and other p
 in the ikiwiki source code, where emailing is done.
 
 --[[tschwinge]]
+
+> One that is in Debian is [[!cpan Email::Send]], which can do SMTP and
+> sendmail and some other methods and falls back through methods until one
+> succeeds. I haven't tried to use it but it looks like a feasable
+> candidate.
+> 
+> I don't much like the idea of supporting a lot of different email sending
+> modules.  --[[Joey]]
+
+OK, so I'll have a look at replacing all email handling with *Email::Send*.
+
+[[!tag patch]]
+*<http://schwinge.homeip.net/~thomas/tmp/ikiwiki-sendmail.patch>*
+
+Remaining TODOs:
+
+  * Resolve TODOs as denoted inside the patch.
+  * Is it worthwhile to use and depend on [[!cpan Return::Value]]
+    just for this bit of functionality?
+  * Debian news file.
+  * ikiwiki news file.
+
+--[[tschwinge]]
+
+
+> BTW, I think you recently sent a patch improving mail logging, but I've
+> lost it. --[[Joey]]
+
+Resent.  --[[tschwinge]]
+
+> Debian now has Mail::Sender, Mail::SendEasy, and Email::Sender
+> (which, according to its dpkg description, "replaces the old and sometimes
+> problematic Email::Send library, which did a decent job at handling very
+> simple email sending tasks, but was not suitable for serious use, for a
+> variety of reasons"). Are any of those any better? It's unfortunate that
+> there doesn't seem to be a clear "best practice"... --[[smcv]]