Changeset 290 for trunk/lib/Utilities.inc.php

Timestamp:

Nov 26, 2007 11:30:12 PM (17 years ago)

Author:

quinn

Message:

Fixed getRemoteAddr() to return only one value of a comma-delimited list.

File:

• trunk/lib/Utilities.inc.php (modified) (1 diff)

trunk/lib/Utilities.inc.php

@@ -849 +849 @@
 {
     $ip = getenv('HTTP_CLIENT_IP');
-    if (empty($ip) || $ip == 'unknown' || $ip == 'localhost' || $ip == '127.0.0.1') {
+    if (in_array($ip, array('', 'unknown', 'localhost', '127.0.0.1'))) {
         $ip = getenv('HTTP_X_FORWARDED_FOR');
-        if (empty($ip) || $ip == 'unknown' || $ip == 'localhost' || $ip == '127.0.0.1') {
+        if (mb_strpos($ip, ',') !== false) {
+            // If HTTP_X_FORWARDED_FOR returns a comma-delimited list of IPs then return the first one (assuming the first is the original).
+            $ips = explode(',', $ip, 2);
+            $ip = $ips[0];
+        }
+        if (in_array($ip, array('', 'unknown', 'localhost', '127.0.0.1'))) {
             $ip = getenv('REMOTE_ADDR');
         }