[GitHub] trafficserver pull request #842: TS-4717: Http2 stack explosion.

2016-08-09 Thread shinrich
Github user shinrich closed the pull request at: https://github.com/apache/trafficserver/pull/842 --- 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 featur

[GitHub] trafficserver pull request #842: TS-4717: Http2 stack explosion.

2016-08-09 Thread shinrich
Github user shinrich commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/842#discussion_r74137828 --- Diff: proxy/http2/Http2ClientSession.h --- @@ -243,6 +243,12 @@ class Http2ClientSession : public ProxyClientSession return "http/2";

[GitHub] trafficserver pull request #842: TS-4717: Http2 stack explosion.

2016-08-09 Thread jpeach
Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/842#discussion_r74106638 --- Diff: proxy/http2/Http2ClientSession.h --- @@ -243,6 +243,12 @@ class Http2ClientSession : public ProxyClientSession return "http/2";

[GitHub] trafficserver pull request #842: TS-4717: Http2 stack explosion.

2016-08-05 Thread shinrich
GitHub user shinrich opened a pull request: https://github.com/apache/trafficserver/pull/842 TS-4717: Http2 stack explosion. Added a common state_process_frame_read method to loop over reading frames while there is data available. The original state_start_frame_read and state_com