Changeset 805 for trunk/Modules/Referrer/Referrer.php

Timestamp:

May 24, 2014, 11:35:47 PM (10 years ago)

Author:

chronos

Message:

• Fixed: Do not log client proxy IP address as remote address because it can be faked and also there can be multiple client proxy addresses.
• Modified: Do not use directly $_SERVER variables REMOTE_ADDR and REQUEST_URI as they are not initialized if script is executed from command line.
• Fixed: Default configuration was not complete.

File:

• trunk/Modules/Referrer/Referrer.php (modified) (1 diff)

trunk/Modules/Referrer/Referrer.php

@@ -40 +40 @@
       if(!in_array($HostName, $this->Excludes))
       {
-        if(!isset($_SERVER['REMOTE_ADDR'])) $IP = 'Konzole';
-          else $IP = addslashes($_SERVER['REMOTE_ADDR']);
-
+        $IP = GetRemoteAddress();
+        
         // Check if client IP is not blocked as spam source
         $DbResult = $this->System->Database->query('SELECT COUNT(*) FROM `Referrer` WHERE `LastIP` = "'.$IP.'" AND (`Visible` = 0)');