Re: [Postfixadmin-devel] Integration with MLM?
Brought to you by:
christian_boltz,
gingerdog
From: Aaron <ar...@gr...> - 2011-01-11 15:28:09
|
On 1/11/2011 4:55 AM, Christian Boltz wrote: > Hello, > > (Was it intentional to reply off-list? If not, feel free to reply to > this mail on the list.) > Indeed, it wasn't intentional, sorry about that. > Am Dienstag, 11. Januar 2011 schrieb Aaron: >> Here's a dumb question, what about integrating some kind of post >> hook when using option #1? ie, some kind of pluggable script spot >> where you could pass something like >> >> external-command.sh [add|remove|modify] aliasname ? > I don't see a real need for script hooks when aliases are created, > modified or deleted. If you have a usecase, tell me ;-) > Other than handling potential MLM integration? I can't think of another use currently, no. There may be additional cases where you'd pipe an alias out to an external command for handling, but I haven't run across them yet. >> I realize that may be a slippery slope, but I could see a few places >> where having a post-action script might have value (like delete >> having an option to tgz the user's folder, etc). > We have that already (for mailboxes and domains), however it's disabled > in the default config. > > grep 'CONF.*script' config.inc.php > // $CONF['mailbox_postcreation_script']='sudo -u courier /usr/local/bin/postfixadmin-mailbox-postcreation.sh'; > // $CONF['mailbox_postedit_script']='sudo -u courier /usr/local/bin/postfixadmin-mailbox-postedit.sh'; > // $CONF['mailbox_postdeletion_script']='sudo -u courier /usr/local/bin/postfixadmin-mailbox-postdeletion.sh'; > // $CONF['domain_postcreation_script']='sudo -u courier /usr/local/bin/postfixadmin-domain-postcreation.sh'; > // $CONF['domain_postdeletion_script']='sudo -u courier /usr/local/bin/postfixadmin-domain-postdeletion.sh'; > That's kind of what I was thinking would be right for aliases. Creating the aliases in postfixadmin first, then running whatever additional commands manually isn't an impossible task, but I have to admit being able to do everything in one place has it's benefits. I can see how it would add complexity, you'd certainly need to add something like "enable external script" as an unselected checkbox to avoid running the external hook on every alias transaction. Would it be difficult to extend the existing script options to include a set for aliases ? (and make it selectable on the alias page?) Thank you for your time, regardless. Aaron |