[GitHub] trafficserver pull request #1600: fix , remove not used array `buf` in `rcv_...

2017-03-22 Thread byronhe
GitHub user byronhe opened a pull request: https://github.com/apache/trafficserver/pull/1600 fix , remove not used array `buf` in `rcv_data_frame()` fix , remove not used array `buf` in `rcv_data_frame()` You can merge this pull request into a Git repository by running: $

[GitHub] trafficserver issue #1600: fix , remove not used array buf in rcv_data_frame...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1600 FreeBSD11 build *successful*! https://ci.trafficserver.apache.org/job/freebsd-github/1786/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1600: fix , remove not used array buf in rcv_data_frame...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1600 Linux build *successful*! https://ci.trafficserver.apache.org/job/linux-github/1679/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1600: fix , remove not used array buf in rcv_data_frame...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1600 Intel CC build *successful*! https://ci.trafficserver.apache.org/job/icc-github/217/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1600: fix , remove not used array buf in rcv_data_frame...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1600 AU check *successful*! https://ci.trafficserver.apache.org/job/autest-github/88/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] trafficserver issue #1600: fix , remove not used array buf in rcv_data_frame...

2017-03-22 Thread bryancall
Github user bryancall commented on the issue: https://github.com/apache/trafficserver/pull/1600 [approve ci] --- 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

[GitHub] trafficserver issue #1600: fix , remove not used array buf in rcv_data_frame...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1600 RAT check *successful*! https://ci.trafficserver.apache.org/job/RAT-github/104/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] trafficserver issue #1600: fix , remove not used array buf in rcv_data_frame...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1600 clang format *successful*! https://ci.trafficserver.apache.org/job/clang-format-github/91/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1583: Set HttpSM::tunnel_handler_post to handle write e...

2017-03-22 Thread maskit
Github user maskit commented on the issue: https://github.com/apache/trafficserver/pull/1583 @oknet The assert has gone. At least it works with H2. Thanks. --- 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

[GitHub] trafficserver pull request #1596: TS-4976: Regularize plugins - output_heade...

2017-03-22 Thread SolidWallOfCode
Github user SolidWallOfCode closed the pull request at: https://github.com/apache/trafficserver/pull/1596 --- 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

[GitHub] trafficserver issue #1600: fix , remove not used array buf in rcv_data_frame...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1600 clang-analyzer build *successful*! https://ci.trafficserver.apache.org/job/clang-analyzer-github/349/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1557: brotli support in gzip plugin

2017-03-22 Thread myraid
Github user myraid commented on the issue: https://github.com/apache/trafficserver/pull/1557 @shukitchan @bryancall can you review the new code. changed enforce_brotli to wildcard pattern matching for urls. Now enforce_brotli will override some of the existing configuration

[GitHub] trafficserver pull request #1600: fix , remove not used array buf in rcv_dat...

2017-03-22 Thread bryancall
Github user bryancall closed the pull request at: https://github.com/apache/trafficserver/pull/1600 --- 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] trafficserver issue #1601: TS-4976: Regularize plugins - protocol

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1601 clang format *successful*! https://ci.trafficserver.apache.org/job/clang-format-github/92/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1601: TS-4976: Regularize plugins - protocol

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1601 RAT check *successful*! https://ci.trafficserver.apache.org/job/RAT-github/105/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] trafficserver issue #1601: TS-4976: Regularize plugins - protocol

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1601 FreeBSD11 build *successful*! https://ci.trafficserver.apache.org/job/freebsd-github/1787/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1602: proxy.config.ssl.server.multicert.exit_on_load_fa...

2017-03-22 Thread mlibbey
GitHub user mlibbey opened an issue: https://github.com/apache/trafficserver/issues/1602 proxy.config.ssl.server.multicert.exit_on_load_fail not documented Not sure when proxy.config.ssl.server.multicert.exit_on_load_fail was added, but its not documented. What does it do?

[GitHub] trafficserver pull request #1601: TS-4976: Regularize plugins - protocol

2017-03-22 Thread SolidWallOfCode
GitHub user SolidWallOfCode opened a pull request: https://github.com/apache/trafficserver/pull/1601 TS-4976: Regularize plugins - protocol Incremental work on #1114. This also updates the documentation on building a protocol plugin. In addition to changes required by the

[GitHub] trafficserver issue #1601: TS-4976: Regularize plugins - protocol

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1601 AU check *successful*! https://ci.trafficserver.apache.org/job/autest-github/89/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] trafficserver pull request #1601: TS-4976: Regularize plugins - protocol

2017-03-22 Thread maskit
Github user maskit commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1601#discussion_r107577489 --- Diff: example/protocol/Protocol.c --- @@ -122,33 +122,29 @@ TSPluginInit(int argc, const char *argv[]) server_port = 4666; if

[GitHub] trafficserver issue #1601: TS-4976: Regularize plugins - protocol

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1601 Linux build *successful*! https://ci.trafficserver.apache.org/job/linux-github/1680/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1601: TS-4976: Regularize plugins - protocol

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1601 Intel CC build *successful*! https://ci.trafficserver.apache.org/job/icc-github/218/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1601: TS-4976: Regularize plugins - protocol

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1601 clang-analyzer build *successful*! https://ci.trafficserver.apache.org/job/clang-analyzer-github/350/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1583: Set HttpSM::tunnel_handler_post to handle write e...

2017-03-22 Thread oknet
Github user oknet commented on the issue: https://github.com/apache/trafficserver/pull/1583 @zwoop @SolidWallOfCode In HttpSM::tunnel_handler_post(), calling tunnel_handler_post_or_put(p) before vc_table.cleanup_all() and tunnel.chain_abort_all(p) to avoid assert that @maskit

[GitHub] trafficserver issue #1583: Set HttpSM::tunnel_handler_post to handle write e...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1583 FreeBSD11 build *successful*! https://ci.trafficserver.apache.org/job/freebsd-github/1784/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1583: Set HttpSM::tunnel_handler_post to handle write e...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1583 clang format *successful*! https://ci.trafficserver.apache.org/job/clang-format-github/90/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1583: Set HttpSM::tunnel_handler_post to handle write e...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1583 AU check *successful*! https://ci.trafficserver.apache.org/job/autest-github/86/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] trafficserver issue #1583: Set HttpSM::tunnel_handler_post to handle write e...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1583 RAT check *successful*! https://ci.trafficserver.apache.org/job/RAT-github/103/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] trafficserver issue #1583: Set HttpSM::tunnel_handler_post to handle write e...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1583 AU check *successful*! https://ci.trafficserver.apache.org/job/autest-github/87/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] trafficserver issue #1583: Set HttpSM::tunnel_handler_post to handle write e...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1583 Linux build *successful*! https://ci.trafficserver.apache.org/job/linux-github/1678/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1583: Set HttpSM::tunnel_handler_post to handle write e...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1583 Intel CC build *successful*! https://ci.trafficserver.apache.org/job/icc-github/216/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1583: Set HttpSM::tunnel_handler_post to handle write e...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1583 clang-analyzer build *successful*! https://ci.trafficserver.apache.org/job/clang-analyzer-github/347/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1583: Set HttpSM::tunnel_handler_post to handle write e...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1583 clang-analyzer build *successful*! https://ci.trafficserver.apache.org/job/clang-analyzer-github/348/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1583: Set HttpSM::tunnel_handler_post to handle write e...

2017-03-22 Thread maskit
Github user maskit commented on the issue: https://github.com/apache/trafficserver/pull/1583 I might be able to reproduce the issue, but only with H2. I couldn't reproduce it with H1. What I did are: - Add sleep(5) into consume() of PostBuffer plugin - Set

[GitHub] trafficserver issue #1583: Set HttpSM::tunnel_handler_post to handle write e...

2017-03-22 Thread oknet
Github user oknet commented on the issue: https://github.com/apache/trafficserver/pull/1583 The vc_table is cleanup_all() and tunnel is chain_abort_all(p) before we call tunnel_handler_post_or_put(p). ``` vc_table.cleanup_all(); tunnel.chain_abort_all(p); ``` we

[GitHub] trafficserver issue #1583: Set HttpSM::tunnel_handler_post to handle write e...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1583 clang format *successful*! https://ci.trafficserver.apache.org/job/clang-format-github/89/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1583: Set HttpSM::tunnel_handler_post to handle write e...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1583 RAT check *successful*! https://ci.trafficserver.apache.org/job/RAT-github/102/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] trafficserver issue #1583: Set HttpSM::tunnel_handler_post to handle write e...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1583 Intel CC build *failed*! https://ci.trafficserver.apache.org/job/icc-github/215/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] trafficserver issue #1583: Set HttpSM::tunnel_handler_post to handle write e...

2017-03-22 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1583 Linux build *failed*! https://ci.trafficserver.apache.org/job/linux-github/1677/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub