[trafficserver] branch master updated: TS-4341 correct warning check

[ja...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

jacksontj pushed a commit to branch master
in repository https://git-dual.apache.org/repos/asf/trafficserver.git

The following commit(s) were added to refs/heads/master by this push:
       new  8795e0b   TS-4341 correct warning check
8795e0b is described below

commit 8795e0b0ddb66073a1948bb1309e007fe63bc323
Author: Thomas Jackson <ja...@gmail.com>
AuthorDate: Fri Apr 15 11:56:14 2016 -0700

    TS-4341 correct warning check
    
    we should only warn users if the queue is enabled-- which is a non-negative number.
---
 proxy/http/HttpConfig.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/proxy/http/HttpConfig.cc b/proxy/http/HttpConfig.cc
index 1ee7ed5..504a385 100644
--- a/proxy/http/HttpConfig.cc
+++ b/proxy/http/HttpConfig.cc
@@ -1165,7 +1165,7 @@ HttpConfig::reconfigure()
   params->oride.origin_max_connections = m_master.oride.origin_max_connections;
   params->oride.origin_max_connections_queue = m_master.oride.origin_max_connections_queue;
   // if origin_max_connections_queue is set without max_connections, it is meaningless, so we'll warn
-  if (params->oride.origin_max_connections_queue &&
+  if (params->oride.origin_max_connections_queue >= 0 &&
       !(params->oride.origin_max_connections || params->origin_min_keep_alive_connections)) {
     Warning("origin_max_connections_queue is set, but neither origin_max_connections nor origin_min_keep_alive_connections are "
             "set, please correct your records.config");

-- 
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].