Re: Antwort: noLog is not respected...

Sorry,  it was from going from 2.149 to 2.2…

This part changed, but some more too…

I can’t read the code good enough….

    if (

        $noLogRe

        && (   ( $this->{mailfrom} && $this->{mailfrom} =~ ( '(' . $noLogReRE . ')' ) )

            || ( $this->{header} =~ ( '(' . $noLogReRE . ')' ) ) )

      ) {

        return 1;

    }

Became

    if ($this->{noLog} || ($noLogRe

        && (( $this->{mailfrom} && $this->{mailfrom} =~ ( '(' . $noLogReRE . ')' ))

        || ( $this->{header} =~ ( '(' . $noLogReRE . ')' )))))

    {

        $this->{noLog} = 1;

        return 1;

    }

There was nothing changed in logging-Code since 2.0.0_1.xx - the upgrade should not be the reason for this! Do you have anything changed in config?

Thomas

As of going from version 2.18 to 2.20 I'm getting this in my log (it's my watchdog)

Jul-2-08 11:15:01 [Worker_1] 192.168.10.130 IP 192.168.10.130 (192.168) matches acceptAllMail
Jul-2-08 11:16:01 [Worker_1] 192.168.10.130 IP 192.168.10.130 (192.168) matches acceptAllMail
Jul-2-08 11:17:02 [Worker_1] 192.168.10.130 IP 192.168.10.130 (192.168) matches acceptAllMail
Jul-2-08 11:18:01 [Worker_1] 192.168.10.130 IP 192.168.10.130 (192.168) matches acceptAllMail
Jul-2-08 11:19:02 [Worker_1] 192.168.10.130 IP 192.168.10.130 (192.168) matches acceptAllMail




-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
Assp-test mailing list
Assp-test@...
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!

[root@fc logs]# cat ../assp.cfg | grep -i nolog

noLog:=192.168.10.130

noLogRe:=

My log tells me it started after restarting ASSP..

I can send you both versions if you don’t have them.

Probably I pointed to the wrong part of the code….

Jean-Pierre,

the code you are pointing to, belongs to "noLogRE" not "noLog"! Please tell me your configuration for this two values!

Thomas

Sorry,  it was from going from 2.149 to 2.2…
 
This part changed, but some more too…
I can’t read the code good enough….
 
 
    if (
        $noLogRe
        && (   ( $this->{mailfrom} && $this->{mailfrom} =~ ( '(' . $noLogReRE . ')' ) )
            || ( $this->{header} =~ ( '(' . $noLogReRE . ')' ) ) )
      ) {
        return 1;
    }
 
Became
 
    if ($this->{noLog} || ($noLogRe
        && (( $this->{mailfrom} && $this->{mailfrom} =~ ( '(' . $noLogReRE . ')' ))
        || ( $this->{header} =~ ( '(' . $noLogReRE . ')' )))))
    {
        $this->{noLog} = 1;
        return 1;
    }
 
Van: assp-test-bounces@... [mailto:assp-test-bounces@...] Namens Thomas Eckardt/eck
Verzonden: woensdag 2 juli 2008 12:00
Aan: ASSP development mailing list
Onderwerp: [Assp-test] Antwort: noLog is not respected...
 

There was nothing changed in logging-Code since 2.0.0_1.xx - the upgrade should not be the reason for this! Do you have anything changed in config?

Thomas

As of going from version 2.18 to 2.20 I'm getting this in my log (it's my watchdog)

Jul-2-08 11:15:01 [Worker_1] 192.168.10.130 IP 192.168.10.130 (192.168) matches acceptAllMail
Jul-2-08 11:16:01 [Worker_1] 192.168.10.130 IP 192.168.10.130 (192.168) matches acceptAllMail
Jul-2-08 11:17:02 [Worker_1] 192.168.10.130 IP 192.168.10.130 (192.168) matches acceptAllMail
Jul-2-08 11:18:01 [Worker_1] 192.168.10.130 IP 192.168.10.130 (192.168) matches acceptAllMail
Jul-2-08 11:19:02 [Worker_1] 192.168.10.130 IP 192.168.10.130 (192.168) matches acceptAllMail




-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
Assp-test mailing list
Assp-test@...
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!-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08_______________________________________________
Assp-test mailing list
Assp-test@...
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!

    mlog($fh,"IP $ip".($ret==1 ? '' : " ($ret)")." matches $re",1) if $ipmatchLogging  && !matchIP($ip,'noLog') && $ret && !$donotmlog;

Isn’t the # in line 17500 the culprit???

    mlog($fh,"IP $ip".($ret==1 ? '' : " ($ret)")." matches $re",1) if $ipmatchLogging && $ret && !$donotmlog; # && ! &matchIP($ip,'noLog');