read and try to understand the code from :
# skip check when RELAYOK
if (!$this->{uhnoprocessing} && (!$this->{relayok} ||
($this->{relayok} && $DoLocalSender))) {
........
to:
.......
} else {
$this->{islocalmailaddress}=localmail($h);
}
It does not help to pick out some single lines.
How ever, feel free to change the code to your needs - if it helps to
solve your problems.
Thomas
Andrew Porter <[email protected]>
12.11.2009 14:09
Bitte antworten an
ASSP development mailing list <[email protected]>
An
ASSP development mailing list <[email protected]>
Kopie
Thema
Re: [Assp-test] Antwort: Re: Antwort: Re: Antwort: Re: Antwort: Re:
Antwort: Re: Validate Local Addresses ASSP 2.0.1(RC 0.5.01)
Thomas Eckardt/eck wrote:
>> Well - an email to [email protected] is a localdomain - so it
>> returns 1.
>
>> Doesn't consider LocalAddresses_flat at all.
>
> What has 'LocalAddresses_flat' to do with that, except that you has
> changed (skipped) a negative check to this list to solve a problem which
> is (at least) related to &localmail at this point of code.
That piece of code sets islocalmailaddress - which determines that the
recipient address is valid and local -
See the code a little further down -
if ( $LocalAddresses_Flat && matchSL( $uh,
'LocalAddresses_Flat' ) ) {
if(matchSL( $uh, 'RejectTheseLocalAddresses' )) {
d("$u$h rejected by bounce address list\n");
} else {
$this->{islocalmailaddress}=1;
d("$u$h validated by flat LocalAddresses list\n");
}
}
Which only positively confirms it's a valid local address - it doesn't
set it negative if it's not. Perhaps this is actually where the problem
lies ? islocalmailaddress is already 1 by here and this doesn't set it
to 0 otherwise.
>
>> Well - an email to [email protected] is a localdomain - so it
>> returns 1.
>
> And what is problem now ? A address is only consider local - if
&localmail
> returns 1 in any case.
It's not a valid local email address. Only a valid domain. I am using
LocalAddresses_Flat to block anything but valid addresses.
------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008
30-Day
trial. Simplify your report design, integration and deployment - and focus
on
what you do best, core application coding. Discover what's new with
Crystal Reports now. http://p.sf.net/sfu/bobj-july
_______________________________________________
Assp-test mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/assp-test
DISCLAIMER:
*******************************************************
This email and any files transmitted with it may be confidential, legally
privileged and protected in law and are intended solely for the use of the
individual to whom it is addressed.
This email was multiple times scanned for viruses. There should be no
known virus in this email!
*******************************************************
------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day
trial. Simplify your report design, integration and deployment - and focus on
what you do best, core application coding. Discover what's new with
Crystal Reports now. http://p.sf.net/sfu/bobj-july
_______________________________________________
Assp-test mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/assp-test