I just saw that this part of code is outdated - sorry! Thomas
Andrew Porter <[email protected]> 12.11.2009 16:07 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: Antwort: Re: Validate Local Addresses ASSP 2.0.1(RC 0.5.01) Thomas Eckardt/eck wrote: > 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. You are missing the point - I have debugged this to pieces and the code I am quoting is the code that sets islocalmailaddress to 1 - not the code you quote. I am not just picking out some random lines - I am picking out the lines that I have traced this down to. Also - the code you are quoting is setting islocalmailaddress from the localmail sub - which returns 1 immediately if the recipient domain is a local domain - which it is. It does nothing with LocalAddresses_Flat because that test is done elsewhere - where I quoted. The code that sets this to 1 is inside the true side of the condition you have quoted - the else is not being reached. Please - I am not trying to be difficult - I am trying my best to explain the situation as I am finding it. I have changed the code and I have it working as I want and need it to but I am concerned that I may have overlooked something and also, if this is a bug - and it seems to be - perhaps it might be useful to other people if a fix was taken on board ? If it is down to configuration perhaps, instead of chanting the "check your configuration" mantra or assuming I am in idiot that deserves only muted prods, you could point to what might be making ASSP skip the fact that the recipient is not in LocalAddresses_flat ? I have asked for help. If I have made a mistake I will gladly own up to it. If you are able to help I would appreciate it. ------------------------------------------------------------------------------ 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
