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

sorber pushed a commit to branch 6.2.x
in repository https://git-dual.apache.org/repos/asf/trafficserver.git

commit 419d1af8355674236d6eb4f5e9b7f584acdc3c5f
Author: Masaori Koshiba <[email protected]>
AuthorDate: Mon May 23 15:28:58 2016 -0700

    TS-4472: Decrement http_current_active_client_connections_stat in HTTP/2
    
    `http_current_active_client_connections_stat` is incremented in HttpSM.
    And if client connection is HTTP/1.1, this is decremented in 
Http1ClientTransaction.
    But if client connection is HTTP/2, this is not decremented anywhere.
    
    (cherry picked from commit 8ed5bf2c76b437903412e3f34c75bda6b8611966)
---
 proxy/http2/Http2Stream.cc | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/proxy/http2/Http2Stream.cc b/proxy/http2/Http2Stream.cc
index e54ed85..137e9e7 100644
--- a/proxy/http2/Http2Stream.cc
+++ b/proxy/http2/Http2Stream.cc
@@ -508,6 +508,11 @@ Http2Stream::destroy()
   // Clean up the write VIO in case of inactivity timeout
   this->do_io_write(NULL, 0, NULL);
 
+  if (m_active) {
+    m_active = false;
+    HTTP_DECREMENT_DYN_STAT(http_current_active_client_connections_stat);
+  }
+
   HTTP2_DECREMENT_THREAD_DYN_STAT(HTTP2_STAT_CURRENT_CLIENT_STREAM_COUNT, 
_thread);
   ink_hrtime end_time = Thread::get_hrtime();
   HTTP2_SUM_THREAD_DYN_STAT(HTTP2_STAT_TOTAL_TRANSACTIONS_TIME, _thread, 
end_time - _start_time);

-- 
To stop receiving notification emails like this one, please contact
"[email protected]" <[email protected]>.

Reply via email to