X-Git-Url: http://gitweb.michael.orlitzky.com/?p=mailshears.git;a=blobdiff_plain;f=doc%2FTODO;h=d80635746772fcc60f6b78f5889b29ad4d0b4e54;hp=864083e9563d5c968f45564f4d36f1bf56891f9d;hb=HEAD;hpb=3f919051ae4eff1431644c5f0d9f511153350c77 diff --git a/doc/TODO b/doc/TODO index 864083e..d806357 100644 --- a/doc/TODO +++ b/doc/TODO @@ -1,22 +1,26 @@ -* Error reporting sucks, and when a domain or user doesn't exist we - should be able to say so. The describe_domain/user functions - should also work better. For plugins that don't implement domains, - we can return a (count of?) list of users, or fall back to the - user deletion descriptions. +* There is essentially no error handling. We report errors, but we + don't fail when we see one. The main reason for this is that we + don't know when each plugin will be run. If the first plugin + encounters an error, we could quit right there. But what if the + third one fails after the first two succeed? We would need some kind + of rollback mechanism. -* Implement "mv". + For "mv", a rollback is conceivable. But with "rm", there's no going + back. Maybe relying on the user to interpret the output and go + fix stuff himself is the best we can do? - Design: +* Add OpenDKIM support. - This should only work from a domain that exists in postfixadmin to a - domain that already exists in postfixadmin. If the target domain - does not exist, we should error out as soon as possible. +* Rename the "dovecot" plugin to "filesystem". - Once we're sure that the target domain does exist, we can hand off - the "mv" operation to the plugins. Some of them might have nothing - to do -- that's fine. +* Implement moving of domains. -* Add convenience methods (e.g. user_exists, domain_exists) that we - can use in the tests to check results. +* The "pretend mode" output is missing the destination information: -* Potentially add OpenDKIM support. + # mailshears mv "test1@example.com" "test2@example.com" + mailshears, 2020-01-30 14:40:09 -0500 (Plugin: MvPlugin) + -------------------------------------------------------- + DavicalMv - Would move user test1@example.com (User not found) to . + DovecotMv - Would move user test1@example.com (/var/spool/mail/vhosts/example.com/test1) to . + PostfixadminMv - Would move user test1@example.com to . + RoundcubeMv - Would move user test1@example.com (User not found) to .