X-Git-Url: https://sipb.mit.edu/gitweb.cgi/ikiwiki.git/blobdiff_plain/1c8b757580fd0254131a5de6032b13837360221b..698aeb20168b7e22a2ce3618a28fdee32ed4a417:/doc/todo/mailnotification.mdwn diff --git a/doc/todo/mailnotification.mdwn b/doc/todo/mailnotification.mdwn deleted file mode 100644 index 858141008..000000000 --- a/doc/todo/mailnotification.mdwn +++ /dev/null @@ -1,54 +0,0 @@ -Should support mail notification of new and changed pages. - - Hmm, should be easy to implement this.. it runs as a svn post-coommit hook - already, so just look at the userdb, svnlook at what's changed, and send - mails to people who have subscribed. - - A few details: - 1. [[Joey]] mentioned that being able to subscribe to globs as well as - explicitly named pages would be desirable. - 2. I think that since we're using Perl on the backend, being able to - let users craft their own arbitrary regexes would be good. - - Joey points out that this is actually a security hole, because Perl - regexes let you embed (arbitrary?) Perl expressions inside them. Yuck! - -(This is not actually true unless you "use re 'eval';", without which -(?{ code }) is disabled for expressions which interpolate variables. -See perldoc re, second paragraph of DESCRIPTION. It's a little iffy -to allow arbitrary regexen, since it's fairly easy to craft a regular -expression that takes unbounded time to run, but this can be avoided -with the use of alarm to add a time limit. Something like - - eval { # catches invalid regexen - no re 'eval'; # to be sure - local $SIG{ALRM} = sub { die }; - alarm(1); - ... stuff involving m/$some_random_variable/ ... - alarm(0); - }; - if ($@) { ... handle the error ... } - -should be safe. --[[WillThompson]]) - - It would also be good to be able to subscribe to all pages except discussion pages or the SandBox: `* !*/discussion !sandobx`, maybe --[[Joey]] - - 3. Of course if you do that, you want to have form processing on the user - page that lets them tune it, and probably choose literal or glob by - default. - - I think that the new globlist() function should do everything you need. - Adding a field to the prefs page will be trivial --[[Joey]] - - The first cut, I suppose, could use one sendmail process to batch-mail all - subscribers for a given page. However, in the long run, I can see users - demanding a bit of feature creep: - - 4. Each user should be able to tune whether they see the actual diff parts or - not. - 5. Each user should be able to set a maximum desired email size. - 6. We might want to support a user-specified shibboleth string that will be - included in the email they receive so they can easily procmail the messages - into a folder. - - --[[BrandenRobinson]]