This is an automated email from the ASF dual-hosted git repository. zwoop pushed a commit to branch 8.1.x in repository https://gitbox.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/8.1.x by this push: new 44fc5c9 Fix a bug that current_active_client_connections doesn't decrease 44fc5c9 is described below commit 44fc5c99677f732b2d7ab9f8bc421d1717686827 Author: Masakazu Kitajo <mas...@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 eee968e..703feb6 100644 --- a/proxy/http2/Http2ConnectionState.cc +++ b/proxy/http2/Http2ConnectionState.cc @@ -1392,6 +1392,7 @@ Http2ConnectionState::release_stream(Http2Stream *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,