X-Git-Url: http://git.vanrenterghem.biz/git.ikiwiki.info.git/blobdiff_plain/2f8123012c9a6ad077b856d7f801346a4d9800fe..93d77673e44c1c4019a2fcbae2e450f095c4426a:/doc/todo/passwordauth:_sendmail_interface.mdwn?ds=sidebyside 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]] +** + +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]]