Armin,

> Could somebody help me to understand/direct why "checks" is working
> as designed with more than internal envelope address.
> Using this with a combination of in-/external envelope
> "dynamic_altermime_disclaimer" is called for internal email as well.
> 
> sub checks {
>   my($self, $conn, $msginfo) = @_;
>   for my $r (@{$msginfo->per_recip_data}) { # $r contains per-recipient data
>     my($is_local) = $r->recip_is_local; # recipient matches 
> @local_domains_maps
>     # don't bother with local outgoing mail!
>     next if $is_local;
>     dynamic_altermime_disclaimer(@_);
>   }
> }


This comment is incompatible with the 'next if' statement following it:
>     # don't bother with local outgoing mail!
>     next if $is_local;

it should say something like:
  # don't bother with inbound or internal-to-internal mail
(or the condition should be changed)

> Using this with a combination of in-/external envelope
> "dynamic_altermime_disclaimer" is called for internal email as well.

As the loop goes through all recipients of a message, the
dynamic_altermime_disclaimer() is called once for each foreign
recipient (i.e. outgoing mail). Is this intended? What does
the subroutine do? Is it capable of applying per-recipient
modifications, or does it affect the whole message for everyone?

  Mark

------------------------------------------------------------------------------
Beautiful is writing same markup. Internet Explorer 9 supports
standards for HTML5, CSS3, SVG 1.1,  ECMAScript5, and DOM L2 & L3.
Spend less time writing and  rewriting code and more time creating great
experiences on the web. Be a part of the beta today
http://p.sf.net/sfu/msIE9-sfdev2dev
_______________________________________________
AMaViS-user mailing list
AMaViS-user@lists.sourceforge.net 
https://lists.sourceforge.net/lists/listinfo/amavis-user 
 Please visit http://www.ijs.si/software/amavisd/ regularly
 For administrativa requests please send email to rainer at openantivirus dot 
org

Reply via email to