Github user gtenev closed the pull request at:
https://github.com/apache/trafficserver/pull/1100
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user gtenev commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/1100#discussion_r83512075
--- Diff: proxy/http2/Http2Stream.cc ---
@@ -267,10 +267,12 @@ Http2Stream::do_io_close(int /* flags */)
// Make sure any trailing end of
Github user gtenev commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/1100#discussion_r83511226
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -1097,6 +1141,7 @@ void
Http2ConnectionState::send_data_frames(Http2Stream *stream)
{
Github user gtenev commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/1100#discussion_r83510837
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -936,30 +940,70 @@ Http2ConnectionState::cleanup_streams()
void
Github user gtenev commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/1100#discussion_r83508421
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -936,30 +940,70 @@ Http2ConnectionState::cleanup_streams()
void
Github user shinrich commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/1100#discussion_r83291135
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -1097,6 +1141,7 @@ void
Http2ConnectionState::send_data_frames(Http2Stream *stream)
{
Github user shinrich commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/1100#discussion_r83291283
--- Diff: proxy/http2/Http2Stream.cc ---
@@ -267,10 +267,12 @@ Http2Stream::do_io_close(int /* flags */)
// Make sure any trailing end of
Github user shinrich commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/1100#discussion_r83290881
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -936,30 +940,70 @@ Http2ConnectionState::cleanup_streams()
void
Github user gtenev commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/1100#discussion_r83279595
--- Diff: iocore/aio/.diags.log.meta ---
@@ -0,0 +1 @@
+creation_time = 1476307057
--- End diff --
removed, committed by mistake
---
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/1100#discussion_r83155537
--- Diff: iocore/aio/.diags.log.meta ---
@@ -0,0 +1 @@
+creation_time = 1476307057
--- End diff --
Do we need this file?
---
If
GitHub user gtenev opened a pull request:
https://github.com/apache/trafficserver/pull/1100
TS-4916: Fix for an H2-infinite-loop deadlock
This is a fix to prevent destroying of the DLL<> structure and the
following iteration over Http2ConnectionState::stream_list to fall into an
11 matches
Mail list logo