Changed: dont reverse req.ips

parse them as-is. im impartial about
the ordering but this spares some CPU. if you
prefer the other way let me know

lib/request.js

@@ -330,7 +330,7 @@ req.__defineGetter__('secure', function(){
  * the "X-Forwarded-For" ip address list.
  *
  * For example if the value were "client, proxy1, proxy2"
- * you would receive the array `["proxy2", "proxy1", "client"]`
+ * you would receive the array `["client", "proxy1", "proxy2"]`
  * where "proxy2" is the furthest down-stream.
  *
  * @return {Array}
@@ -341,7 +341,7 @@ req.__defineGetter__('ips', function(){
   var trustProxy = this.app.enabled('trust proxy');
   var val = this.get('X-Forwarded-For');
   return trustProxy && val
-    ? val.split(/ *, */).reverse()
+    ? val.split(/ *, */)
     : [];
 });
 

test/req.ips.js

@@ -18,7 +18,7 @@ describe('req', function(){
           request(app)
           .get('/')
           .set('X-Forwarded-For', 'client, p1, p2')
-          .expect('["p2","p1","client"]', done);
+          .expect('["client","p1","p2"]', done);
         })
       })