[GitHub] trafficserver pull request #1614: TS-4976: Regularize plugins - query_remap

Github user SolidWallOfCode closed the pull request at: https://github.com/apache/trafficserver/pull/1614 --- 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 #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1616 AU check *successful*! https://ci.trafficserver.apache.org/job/autest-github/113/ --- 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

Github user myraid commented on the issue: https://github.com/apache/trafficserver/pull/1557 @bryancall added error messages. > enforce-brotli /this/*.js is introduced to selectively turn on brotli compression for a particular path with minimal changes to the

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

Github user SolidWallOfCode commented on the issue: https://github.com/apache/trafficserver/pull/1616 Dang, that's not going to work either. Because it's a prefix it's not wrong if the incoming tag is shorter than the internal tag. --- If your project is set up for it, you can reply

[GitHub] trafficserver issue #1615: Doc: Update plugin stats documentation.

Github user jpeach commented on the issue: https://github.com/apache/trafficserver/pull/1615 > On Mar 28, 2017, at 2:08 PM, Alan M. Carroll wrote: > > I'll see what I can do, @jpeach. This already includes code from the example/statistic

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1615: Doc: Update plugin stats documentation.

Github user SolidWallOfCode commented on the issue: https://github.com/apache/trafficserver/pull/1615 I'll see what I can do, @jpeach. This already includes code from the `example/statistic` plugin. In practice double registering is normally done by avoiding the double register path

Jenkins build is back to normal : ubuntu_16_04-master » gcc,ubuntu_16_04,debug #649

See

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

[GitHub] trafficserver issue #1616: Fix string compare for protocol tag logic in Unix...

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

Jenkins build is back to normal : fedora_25-master » gcc,fedora_25,release #144

See

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

See

[GitHub] trafficserver pull request #1616: Fix string compare for protocol tag logic ...

GitHub user SolidWallOfCode opened a pull request: https://github.com/apache/trafficserver/pull/1616 Fix string compare for protocol tag logic in UnixNetVConnection. Switch to using the `StringView` based `strcmp` which handles length issues. The current code is a bit weak on that

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

See

Jenkins build is back to normal : fedora_25-master » gcc,fedora_25,debug #144

See

Jenkins build is back to normal : fedora_24-master » gcc,fedora_24,debug #417

See

[GitHub] trafficserver issue #1615: Doc: Update plugin stats documentation.

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

[GitHub] trafficserver issue #1615: Doc: Update plugin stats documentation.

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

[GitHub] trafficserver issue #1615: Doc: Update plugin stats documentation.

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

[GitHub] trafficserver issue #1615: Doc: Update plugin stats documentation.

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

[GitHub] trafficserver issue #1615: Doc: Update plugin stats documentation.

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1615 RAT check *successful*! https://ci.trafficserver.apache.org/job/RAT-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 #1615: Doc: Update plugin stats documentation.

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1615 AU check *failed*! https://ci.trafficserver.apache.org/job/autest-github/110/ --- 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 #1615: Doc: Update plugin stats documentation.

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

[GitHub] trafficserver pull request #1615: Doc: Update plugin stats documentation.

GitHub user SolidWallOfCode opened a pull request: https://github.com/apache/trafficserver/pull/1615 Doc: Update plugin stats documentation. I wanted to tell a co-worked to use the plugin stats but noticed they were not in fact well documented. You can merge this pull request into

[GitHub] trafficserver issue #1609: Adding a stat page for HTTP connection count

Github user bgaff commented on the issue: https://github.com/apache/trafficserver/pull/1609 The only concern I would have is holding the mutex that entire time but given this will only be a http endpoint I think it's fine. @bryancall if you don't have any issues w/ this I'll land it.

[GitHub] trafficserver pull request #1608: FTP Support

Github user dragon512 commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1608#discussion_r108465171 --- Diff: ci/tsqa/tests/test_forward_proxy.py --- @@ -0,0 +1,123 @@ +''' --- End diff -- for the test engine what I have at the

[GitHub] trafficserver pull request #1608: FTP Support

Github user clearswift commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1608#discussion_r108464212 --- Diff: ci/tsqa/tests/test_forward_proxy.py --- @@ -0,0 +1,123 @@ +''' --- End diff -- Is there documentation for the new AU

[GitHub] trafficserver pull request #1608: FTP Support

Github user clearswift commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1608#discussion_r108463999 --- Diff: proxy/http/HttpConfig.h --- @@ -824,6 +824,8 @@ struct HttpConfigParams : public ConfigInfo { MgmtInt

[GitHub] trafficserver pull request #1608: FTP Support

Github user zwoop commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1608#discussion_r108463799 --- Diff: ci/tsqa/tests/test_forward_proxy.py --- @@ -0,0 +1,123 @@ +''' --- End diff -- If at all possible, can you convert this to

[GitHub] trafficserver issue #1613: FTP Support

Github user clearswift closed the issue at: https://github.com/apache/trafficserver/issues/1613 --- 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 #1613: FTP Support

Github user clearswift commented on the issue: https://github.com/apache/trafficserver/issues/1613 Closes as a duplicate. See PR #1608 --- 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

[GitHub] trafficserver pull request #1608: FTP Support

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

[GitHub] trafficserver pull request #1608: FTP Support

Github user zwoop commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1608#discussion_r108462430 --- Diff: proxy/http/HttpConfig.h --- @@ -824,6 +824,8 @@ struct HttpConfigParams : public ConfigInfo { MgmtInt

[GitHub] trafficserver issue #1614: TS-4976: Regularize plugins - query_remap

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

[GitHub] trafficserver issue #1614: TS-4976: Regularize plugins - query_remap

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

[GitHub] trafficserver issue #1614: TS-4976: Regularize plugins - query_remap

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

[GitHub] trafficserver issue #1614: TS-4976: Regularize plugins - query_remap

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

[GitHub] trafficserver issue #1614: TS-4976: Regularize plugins - query_remap

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

[GitHub] trafficserver issue #1614: TS-4976: Regularize plugins - query_remap

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

[GitHub] trafficserver issue #1614: TS-4976: Regularize plugins - query_remap

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1614 clang format *successful*! https://ci.trafficserver.apache.org/job/clang-format-github/112/ --- 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

Github user bryancall commented on the issue: https://github.com/apache/trafficserver/pull/1557 @myraid There was an issue with my build. I got it working. It would would be great to have debug and error warnings if you haven't built it with brotli and have it configured that way.

[GitHub] trafficserver pull request #1614: TS-4976: Regularize plugins - query_remap

GitHub user SolidWallOfCode opened a pull request: https://github.com/apache/trafficserver/pull/1614 TS-4976: Regularize plugins - query_remap Incremental work on #1114. This also include updates to the documentation about `query_remap`. You can merge this pull request into a Git

[GitHub] trafficserver pull request #1612: Changed some of the HTTP/2 enums to enum c...

Github user bryancall closed the pull request at: https://github.com/apache/trafficserver/pull/1612 --- 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 pull request #1611: Convert HPACK enums to enum classes

Github user bryancall closed the pull request at: https://github.com/apache/trafficserver/pull/1611 --- 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 pull request #1604: TS-4976: Regularize plugins - protocol_sta...

Github user SolidWallOfCode closed the pull request at: https://github.com/apache/trafficserver/pull/1604 --- 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 #1604: TS-4976: Regularize plugins - protocol_stack

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

[GitHub] trafficserver issue #1604: TS-4976: Regularize plugins - protocol_stack

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

[GitHub] trafficserver issue #1604: TS-4976: Regularize plugins - protocol_stack

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

[GitHub] trafficserver issue #1604: TS-4976: Regularize plugins - protocol_stack

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

[GitHub] trafficserver issue #1604: TS-4976: Regularize plugins - protocol_stack

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

[GitHub] trafficserver issue #1604: TS-4976: Regularize plugins - protocol_stack

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

[GitHub] trafficserver issue #1604: TS-4976: Regularize plugins - protocol_stack

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1604 RAT check *successful*! https://ci.trafficserver.apache.org/job/RAT-github/125/ --- 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 #1594: Early intervention APIs

Github user SolidWallOfCode closed the pull request at: https://github.com/apache/trafficserver/pull/1594 --- 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 #1613: FTP Support

GitHub user clearswift opened an issue: https://github.com/apache/trafficserver/issues/1613 FTP Support At the moment TS blocks requests for FTP URLs with an 'Unsupported Protocol' error message. This precludes TS from simply forwarding requests for FTP URLs to an upstream