#55 More flexible vacation module regarding mailsever definition

closed-fixed
nobody
Vacation (11)
5
2013-12-15
2009-06-27
Stevan Bajic
No

The currently used Mail::sendmail module is very limited regarding the configuration of SMTP server data. You can't define:
- SMTP port
- SMTP username
- SMTP password
- SMTP authentication

I am using Postfix as my MTA and using virtual vacation is a no-go for me since I do sender mismatch checking and with the currently used Mail::sendmail in virtual vacation I can use virtual vacation on my setup.

I would like to replace the Mail::sendmail module with Mail::Sender which is offering much more flexibility then Mail::sendmail. Should I send a patch to virtual vacation v4 using Mail::Sender for sending mails? Is the above mentioned functionality useful for other people? How high are the chances to get such a patch merged into Postfix.Admin?

Discussion

  • GingerDog
    GingerDog
    2009-06-27

    Hi, yes I'm more than happy to see a patch, and would almost certainly merge it

    thanks
    David

     
  • Stevan Bajic
    Stevan Bajic
    2009-06-27

    Hallo David

    Here a very quick shot of a patch. From my viewpoint many things could be made better. The whole enconding part could be made less depended on external modules since Mail::Sender has them built in. I will anyway post the patch but work later on it to make it more flexible.

    Kind Regards from Switzerland

    Steve

     
  • GingerDog
    GingerDog
    2009-06-27

    Cool. Thanks for the quick reply. I'm no perl pro, so I'm sure much could be improved. I hadn't heard of mail::sender before either. Keep the patches coming, will try and read/merge this evening

     
  • Charles
    Charles
    2009-06-27

    Can't you just override the sender_mismatch restriction for the vacation service? Or maybe you can't do that with pipes?

    Anyway, this sounds good... thanks for the contrib, Steve!

     
  • GingerDog
    GingerDog
    2009-06-29

    merged into trunk;

     
    • status: open --> closed-fixed
    • Group: --> SVN (please specify revision!)
     
  • finally closing - as the previous comment says, this was merged into SVN trunk looooong time ago ;-)