[trafficserver] branch master updated: Fix proxy.process.http.current_client_connections (#7056)

[sh...@apache.org]

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

shinrich pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new d3875eb  Fix proxy.process.http.current_client_connections (#7056)
d3875eb is described below

commit d3875eb4e016e042c25a2f98cb52ed1638dfb02b
Author: Susan Hinrichs <sh...@yahoo-inc.com>
AuthorDate: Thu Jul 30 09:25:24 2020 -0500

    Fix proxy.process.http.current_client_connections (#7056)
---
 proxy/http/Http1ClientSession.cc | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/proxy/http/Http1ClientSession.cc b/proxy/http/Http1ClientSession.cc
index 2005be4..9177c0e 100644
--- a/proxy/http/Http1ClientSession.cc
+++ b/proxy/http/Http1ClientSession.cc
@@ -261,12 +261,8 @@ Http1ClientSession::do_io_close(int alerrno)
   } else {
     HttpSsnDebug("[%" PRId64 "] session closed", con_id);
     HTTP_SUM_DYN_STAT(http_transactions_per_client_con, transact_count);
-    if (read_state != HCS_ACTIVE_READER) {
-      // donot double decrement
-      HTTP_DECREMENT_DYN_STAT(http_current_client_connections_stat);
-    }
-    read_state    = HCS_CLOSED;
-    conn_decrease = false;
+    read_state = HCS_CLOSED;
+
     // Can go ahead and close the netvc now, but keeping around the session object
     // until all the transactions are closed
     if (_vc) {