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 2020/05/05 04:16:28 UTC
[trafficserver] branch 9.0.x updated: Fix a bug that
current_active_client_connections doesn't decrease
This is an automated email from the ASF dual-hosted git repository.
zwoop pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/9.0.x by this push:
new 32eec91 Fix a bug that current_active_client_connections doesn't decrease
32eec91 is described below
commit 32eec91bb38450612140047f18a113d48e18f0fe
Author: Masakazu Kitajo <ma...@apache.org>
AuthorDate: Fri May 1 16:43:47 2020 +0900
Fix a bug that current_active_client_connections doesn't decrease
The metric was not decremented when a connection is closed by graceful
shutdown (GOAWAY frame with stream ID 2^31-1).
(cherry picked from commit 81f750ec3988f7a862ac5b66846530aa9f02d363)
---
proxy/http2/Http2ConnectionState.cc | 1 +
1 file changed, 1 insertion(+)
diff --git a/proxy/http2/Http2ConnectionState.cc b/proxy/http2/Http2ConnectionState.cc
index 5386e5a..e8d59a7 100644
--- a/proxy/http2/Http2ConnectionState.cc
+++ b/proxy/http2/Http2ConnectionState.cc
@@ -1406,6 +1406,7 @@ Http2ConnectionState::release_stream()
// or we can use a local variable to do it.
// ua_session = nullptr;
} else if (shutdown_state == HTTP2_SHUTDOWN_IN_PROGRESS && fini_event == nullptr) {
+ ua_session->clear_session_active();
fini_event = this_ethread()->schedule_imm_local((Continuation *)this, HTTP2_SESSION_EVENT_FINI);
} else if (ua_session->is_active()) {
// If the number of clients is 0, HTTP2_SESSION_EVENT_FINI is not received or sent, and ua_session is active,