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

2017-04-03 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1557 clang-analyzer build *successful*! https://ci.trafficserver.apache.org/job/clang-analyzer-github/388/ --- 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-04-03 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1557 Intel CC build *successful*! https://ci.trafficserver.apache.org/job/icc-github/256/ --- If your project is set up for it, you can reply to this email and have your reply appear on

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

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

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

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

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

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

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

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

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

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

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

2017-04-03 Thread bryancall
Github user bryancall commented on the issue: https://github.com/apache/trafficserver/pull/1557 [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 #1624: Plugin debugging feature

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

[GitHub] trafficserver issue #1624: Plugin debugging feature

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

[GitHub] trafficserver issue #1624: Plugin debugging feature

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

[GitHub] trafficserver issue #1624: Plugin debugging feature

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

[GitHub] trafficserver issue #1624: Plugin debugging feature

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

[GitHub] trafficserver issue #1624: Plugin debugging feature

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

[GitHub] trafficserver issue #1624: Http transaction debugging feature

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

Jenkins build is back to normal : ubuntu_16_04-master » gcc,ubuntu_16_04,release #655

2017-04-03 Thread jenkins
See

Jenkins build is back to normal : fedora_24-master » gcc,fedora_24,release #423

2017-04-03 Thread jenkins
See

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

2017-04-03 Thread bryancall
Github user bryancall commented on the issue: https://github.com/apache/trafficserver/pull/1557 [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 #1557: brotli support in gzip plugin

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

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

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

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

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

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

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

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

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

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

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

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

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

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

2017-04-03 Thread bryancall
Github user bryancall commented on the issue: https://github.com/apache/trafficserver/pull/1557 [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 #1557: brotli support in gzip plugin

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

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

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

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

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

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

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

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

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

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

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

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

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

[GitHub] trafficserver issue #1624: Http transaction debugging feature

2017-04-03 Thread chenglongwei
Github user chenglongwei commented on the issue: https://github.com/apache/trafficserver/pull/1624 This debugging feature is mainly used to debug plugins. I also modified the title to avoid confusedness. This debugging requires to know 'before or after' 'what hook id', 'what

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

2017-04-03 Thread bryancall
Github user bryancall commented on the issue: https://github.com/apache/trafficserver/pull/1557 [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 #1624: Http transaction debugging feature

2017-04-03 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1624 Yes, I'm very much -1 on adding new plugin APIs and specially the loading of a .so via records.config. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] trafficserver issue #1568: TS-4195: double free on exit

2017-04-03 Thread bryancall
Github user bryancall commented on the issue: https://github.com/apache/trafficserver/pull/1568 Looks good --- 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,

[GitHub] trafficserver pull request #1568: TS-4195: double free on exit

2017-04-03 Thread bryancall
Github user bryancall closed the pull request at: https://github.com/apache/trafficserver/pull/1568 --- 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 #1626: enable proxy.config.http.negative_revalidating_en...

2017-04-03 Thread mlibbey
Github user mlibbey commented on the issue: https://github.com/apache/trafficserver/issues/1626 Don't get this sentence: > If it is enabled, than caching the negative response, the current (possibly stale) content is preserved and served. I think this should only be

[GitHub] trafficserver issue #1624: Http transaction debugging feature

2017-04-03 Thread bryancall
Github user bryancall commented on the issue: https://github.com/apache/trafficserver/pull/1624 It would be better to make these normal transaction hooks and use the TSHttpHookAdd() function to add the hook. Then you can use the normal plugin.config to load the plugin. --- If

[GitHub] trafficserver issue #1626: enable proxy.config.http.negative_revalidating_en...

2017-04-03 Thread SolidWallOfCode
Github user SolidWallOfCode commented on the issue: https://github.com/apache/trafficserver/issues/1626 What about this - .. ts:cv:: CONFIG proxy.config.http.negative_revalidating_enabled INT 0 :reloadable: :overridable: Negative revalidating allows

[GitHub] trafficserver issue #1629: Assertion in state_read_server_response_header (v...

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

[GitHub] trafficserver issue #1629: Assertion in state_read_server_response_header (v...

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

[GitHub] trafficserver issue #1629: Assertion in state_read_server_response_header (v...

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

[GitHub] trafficserver issue #1629: Assertion in state_read_server_response_header (v...

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

[GitHub] trafficserver issue #1629: Assertion in state_read_server_response_header (v...

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

[GitHub] trafficserver issue #1629: Assertion in state_read_server_response_header (v...

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

[GitHub] trafficserver issue #1629: Assertion in state_read_server_response_header (v...

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

[GitHub] trafficserver issue #1629: Assertion in state_read_server_response_header (v...

2017-04-03 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1629 [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 so,

[GitHub] trafficserver pull request #1620: Adds a new condition, %{IP:}

2017-04-03 Thread zwoop
Github user zwoop closed the pull request at: https://github.com/apache/trafficserver/pull/1620 --- 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 feature

[GitHub] trafficserver issue #1630: ATS Lua could support shared dictionary just like...

2017-04-03 Thread scw00
GitHub user scw00 opened an issue: https://github.com/apache/trafficserver/issues/1630 ATS Lua could support shared dictionary just like nginx To replace nginx, ATS Lua could support shared dictionary just like nginx. --- If your project is set up for it, you can reply

[GitHub] trafficserver pull request #1629: Assertion in state_read_server_response_he...

2017-04-03 Thread scw00
GitHub user scw00 opened a pull request: https://github.com/apache/trafficserver/pull/1629 Assertion in state_read_server_response_header (v7.1.0) In my opinion, the EPOLLERR always with read.enabled or write.enabled(otherwise, there is no sm could handle this event, we just

[GitHub] trafficserver pull request #1608: FTP Support

2017-04-03 Thread clearswift
Github user clearswift commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1608#discussion_r109416571 --- Diff: proxy/http/HttpConfig.cc --- @@ -1129,6 +1129,8 @@ HttpConfig::startup() // Local Manager

[GitHub] trafficserver pull request #1628: Assertion in state_read_server_response_he...

2017-04-03 Thread scw00
Github user scw00 closed the pull request at: https://github.com/apache/trafficserver/pull/1628 --- 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 feature

[GitHub] trafficserver pull request #1628: Assertion in state_read_server_response_he...

2017-04-03 Thread scw00
GitHub user scw00 opened a pull request: https://github.com/apache/trafficserver/pull/1628 Assertion in state_read_server_response_header (v7.1.0) In my opinion, the EPOLLERR always with read.enabled or write.enabled(otherwise, there is no sm could handle this event, we just

[GitHub] trafficserver issue #1531: Assertion in state_read_server_response_header (v...

2017-04-03 Thread scw00
Github user scw00 commented on the issue: https://github.com/apache/trafficserver/issues/1531 There are reasons for this creash: 1、ATS try to connect the origin server, but receive RST package. So ATS try again until retry_times == 0. It will clean the server_entry (mark the VC