Menu

#28 Patch for Courier for Amavis CVS

closed-accepted
None
5
2004-01-19
2003-02-22
Jim Gifford
No

This patch will add support for Courier MTA.

Patch changes amavis/checkaccount.in to sense
courier's aliases file and appropriate command to update
it.

Patch changes amavis/Makefile.am to make
the /var/amavis and /var/virusmails to chmod 777 since
courier's maildrop runs as the user, this change only
happens if the courier MTA is selected.

Patch adds amavis/courier_args
amavis/courier_init /amavis/courier_send specific to the
courier mta.

Patch changes amavis/configure.in add the --enable-
courier option

Patch adds amavis/README.courier adds directions for
configuration the courier mta

Discussion

  • Lars Hecking

    Lars Hecking - 2003-02-24
    • assigned_to: nobody --> lhecking
     
  • Lars Hecking

    Lars Hecking - 2003-02-24

    Logged In: YES
    user_id=28904

    This looks quite interesting!

    Unfortunately, I cannot use this for amavisd as it is because
    it requires that amavis prints the mail message to STDOUT (this
    is how the xfilter facility works, right?)

    Would it be possible to set this up similar to the qmail
    support in
    amavis? I.e. install amavis as maildrop, and rename maildrop to
    maildrop-real. Then, instead of printing to STDOUT, let amavis
    open a pipe to maildrop-real. How does that sound?

     
  • Lars Hecking

    Lars Hecking - 2003-02-24

    Logged In: YES
    user_id=28904

    Or, even better, set DEFAULTDELIVERY to |/usr/sbin/amavis,
    and just let amavis open a pipe to maildrop - no renaming
    necessary.

     
  • Jim Gifford

    Jim Gifford - 2003-02-24

    Logged In: YES
    user_id=492775

    The DefaultDelivery variable in Courierd is used to tell courier
    where to deliver the mail to. The normail setting is
    DefaultDelivery = ./Maildir, we override that when we use my
    amavis patch with the command
    DefaultDelivery="| /usr/bin/maildrop", if We put amavis in that,
    amavis will have to put he email in the correct directory. That
    could be a big hassle.

     
  • Lars Hecking

    Lars Hecking - 2003-02-25

    Logged In: YES
    user_id=28904

    Not if amavis calls maildrop, I would think. Like in
    mta/exim_send.

    $LDA="/usr/bin/maildrop";
    @LDAARGS=(); #??

     
  • Jim Gifford

    Jim Gifford - 2003-02-25

    Logged In: YES
    user_id=492775

    If you have an idea, I tried a couple of different things, but it
    bypasses the maildroprc file which contains all of the system
    wide rules.

     
  • Lars Hecking

    Lars Hecking - 2003-02-25

    Logged In: YES
    user_id=28904

    As I'm not familiar with courier, I can't really make a
    suggestion :-/

    My motivation is to keep the code in amavis-perl and amavisd
    as much in sync as possible. maildrop's xfilter stuff cannot be
    used with amavisd because the client passes the message on to
    amavisd instead of filtering it to stdout.

    For the time being, I would like to make your patch available on
    our web site and announce it on amavis-user, maybe some
    other folks have an idea.

     
  • Jim Gifford

    Jim Gifford - 2003-02-26

    Logged In: YES
    user_id=492775

    I think I have a solution that should work with amavis daemon.
    I have posted a new patch that should do the trick.

     
  • Lars Hecking

    Lars Hecking - 2003-03-03

    Logged In: YES
    user_id=28904

    It won't. You can't let the daemon output stuff to STDOUT.
    amavisd can output only through pipes or SMTP.

     
  • Jim Gifford

    Jim Gifford - 2003-07-14

    Updated Amavis Courier Patch

     
  • Jim Gifford

    Jim Gifford - 2003-07-14

    Logged In: YES
    user_id=492775

    Updated Courier Patch. Fixed a few typos.
    Delete old patches.

     
  • Jim Gifford

    Jim Gifford - 2003-07-14

    Logged In: YES
    user_id=492775

    Updated Courier Patch. Fixed a few typos.
    Delete old patches.

     
  • Jim Gifford

    Jim Gifford - 2003-07-14

    Logged In: YES
    user_id=492775

    Updated Courier Patch. Fixed a few typos.
    Delete old patches.

     
  • Lars Hecking

    Lars Hecking - 2004-01-19
    • status: open --> closed-accepted
     
  • Lars Hecking

    Lars Hecking - 2004-01-19

    Logged In: YES
    user_id=28904

    Patch accepted for cvs. Support for amavisd will require
    major rewrite,
    which is planned for version 0.2.

     

Log in to post a comment.