[GitHub] trafficserver pull request: TS-3752: Accept Large Header

2015-08-15 Thread masaori335
Github user masaori335 closed the pull request at: https://github.com/apache/trafficserver/pull/276 --- 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 feat

[GitHub] trafficserver pull request: TS-3752: Accept Large Header

2015-08-15 Thread masaori335
Github user masaori335 commented on the pull request: https://github.com/apache/trafficserver/pull/276#issuecomment-131470073 OK, I'll close. This Pull-Request is merged as 00ce2f1113baa9485262695a66ee67a08fc5d121. --- If your project is set up for it, you can reply to this email

[GitHub] trafficserver pull request: TS-3752: Accept Large Header

2015-08-15 Thread bryancall
Github user bryancall commented on the pull request: https://github.com/apache/trafficserver/pull/276#issuecomment-131468753 I committed this to master, but didn't add This closes #276. Can you please close the pull request? --- If your project is set up for it, you can reply to thi

[GitHub] trafficserver pull request: TS-3752: Accept Large Header

2015-08-14 Thread bryancall
Github user bryancall commented on the pull request: https://github.com/apache/trafficserver/pull/276#issuecomment-131255416 I have been testing with all the recent changes to HTTP/2 and this patch in production and everything looks good. --- If your project is set up for it, you can

[GitHub] trafficserver pull request: TS-3752: Accept Large Header

2015-08-13 Thread masaori335
Github user masaori335 commented on the pull request: https://github.com/apache/trafficserver/pull/276#issuecomment-130907474 Rebased a5818b350d48ea14598d1c876124b393a936f3c9 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] trafficserver pull request: TS-3752: Accept Large Header

2015-08-13 Thread bryancall
Github user bryancall commented on the pull request: https://github.com/apache/trafficserver/pull/276#issuecomment-130838839 Looks good so far, I am going to test it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

[GitHub] trafficserver pull request: TS-3752: Accept Large Header

2015-08-12 Thread masaori335
Github user masaori335 commented on the pull request: https://github.com/apache/trafficserver/pull/276#issuecomment-130510960 Result of h2spec with this patch. ``` ... 68 tests, 65 passed, 1 skipped, 2 failed

[GitHub] trafficserver pull request: TS-3752: Accept Large Header

2015-08-11 Thread masaori335
GitHub user masaori335 opened a pull request: https://github.com/apache/trafficserver/pull/276 TS-3752: Accept Large Header Fix [TS-3752](https://issues.apache.org/jira/browse/TS-3752). Approach: Collect all Header Block Fragments before decode with HPACK. You can merge this pul