You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2020/05/01 07:59:23 UTC
[GitHub] [trafficserver] maskit opened a new pull request #6729: Fix a bug that current_active_client_connections doesn't decrease
maskit opened a new pull request #6729:
URL: https://github.com/apache/trafficserver/pull/6729
`current_active_client_connections` was not decremented when a connection is closed with graceful shutdown (GOAWAY frame with stream ID 2^31-1).
This bug was perhaps introduced by #4936 since `clear_session_active` was originally always called regardless of graceful shutdown.
This fix would fix a part of #6671 but maybe not completely. There's no fix for *leak* on this PR.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] zwoop commented on pull request #6729: Fix a bug that current_active_client_connections doesn't decrease
Posted by GitBox <gi...@apache.org>.
zwoop commented on pull request #6729:
URL: https://github.com/apache/trafficserver/pull/6729#issuecomment-623848797
Cherry-picked to v9.0.x branch.
Cherry-picked to 8.1.x
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] jvgutierrez commented on pull request #6729: Fix a bug that current_active_client_connections doesn't decrease
Posted by GitBox <gi...@apache.org>.
jvgutierrez commented on pull request #6729:
URL: https://github.com/apache/trafficserver/pull/6729#issuecomment-622415197
interesting, thanks @maskit :)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] shinrich commented on pull request #6729: Fix a bug that current_active_client_connections doesn't decrease
Posted by GitBox <gi...@apache.org>.
shinrich commented on pull request #6729:
URL: https://github.com/apache/trafficserver/pull/6729#issuecomment-622409649
[approve ci autest]
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org