SslConnection should implement getBytesIn()/getBytesOut() #5931
Assignees
Labels
Sponsored
This issue affects a user with a commercial support agreement
Comments
sbordet
added
Sponsored
sbordet
added a commit
that referenced
this issue
May 30, 2021
Signed-off-by: Simone Bordet <[email protected]>
sbordet
added a commit
that referenced
this issue
May 30, 2021
Make sure the connection is closed before the assertions. Signed-off-by: Simone Bordet <[email protected]>
sbordet
added a commit
that referenced
this issue
May 30, 2021
Fixed CountDownLatch count. Signed-off-by: Simone Bordet <[email protected]>
sbordet
added a commit
that referenced
this issue
May 30, 2021
Fixed CountDownLatch count again. Signed-off-by: Simone Bordet <[email protected]>
sbordet
added a commit
that referenced
this issue
May 31, 2021
Updates after review. Signed-off-by: Simone Bordet <[email protected]>
sbordet
added a commit
that referenced
this issue
Jun 1, 2021
Updated ConnectionStatistics to report both the stats of all connections, and the stats grouped by connection class. Signed-off-by: Simone Bordet <[email protected]>
sbordet
added a commit
that referenced
this issue
Jun 3, 2021
Updates after review. Signed-off-by: Simone Bordet <[email protected]>
sbordet
added a commit
that referenced
this issue
Jun 3, 2021
The TimeoutException message always references the original timeout. Signed-off-by: Simone Bordet <[email protected]>
sbordet
added a commit
that referenced
this issue
Jun 3, 2021
…(). (#6335) * Fixes #5931 - SslConnection should implement getBytesIn()/getBytesOut(). Updated ConnectionStatistics to report both the stats of all connections, and the stats grouped by connection class. Signed-off-by: Simone Bordet <[email protected]>
sbordet
added a commit
that referenced
this issue
Jun 3, 2021
…(). (#6335) Updated ConnectionStatistics to report both the stats of all connections, and the stats grouped by connection class. Signed-off-by: Simone Bordet <[email protected]> (cherry picked from commit f902d12)
sbordet
added a commit
that referenced
this issue
Jun 3, 2021
…(). (#6335) Updated ConnectionStatistics to report both the stats of all connections, and the stats grouped by connection class. Signed-off-by: Simone Bordet <[email protected]> (cherry picked from commit f902d12)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Jetty version
9.4.x
Description
SslConnectiondoes not implementgetBytesIn()/getBytesOut()but it should, so that load test reports are accurate in showing how many bytes traveled through the network.Using
IncludeExcludeConnectionStatisticswould allow to select what connection needs to be tracked to avoid double-reporting the bytes.The text was updated successfully, but these errors were encountered: