You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@trafficserver.apache.org by zw...@apache.org on 2011/03/08 22:06:50 UTC

svn commit: r1079542 - /trafficserver/traffic/trunk/proxy/logging/LogFilter.cc

Author: zwoop
Date: Tue Mar  8 21:06:50 2011
New Revision: 1079542

URL: http://svn.apache.org/viewvc?rev=1079542&view=rev
Log:
TS-691 LogFilter not working for "int" types

Review: Leif
Author: Eric Connell

Modified:
    trafficserver/traffic/trunk/proxy/logging/LogFilter.cc

Modified: trafficserver/traffic/trunk/proxy/logging/LogFilter.cc
URL: http://svn.apache.org/viewvc/trafficserver/traffic/trunk/proxy/logging/LogFilter.cc?rev=1079542&r1=1079541&r2=1079542&view=diff
==============================================================================
--- trafficserver/traffic/trunk/proxy/logging/LogFilter.cc (original)
+++ trafficserver/traffic/trunk/proxy/logging/LogFilter.cc Tue Mar  8 21:06:50 2011
@@ -452,7 +452,8 @@ bool LogFilterInt::toss_this_entry(LogAc
   int64_t value;
 
   m_field->marshal(lad, (char *) &value);
-  value = ntohl(value);
+  // This used to do an ntohl() on value, but that breaks various filters.
+  // Long term we should move IPs to their own log type.
 
   // we don't use m_operator because we consider all operators to be
   // equivalent to "MATCH" for an integer field