[GitHub] trafficserver pull request #781: TS-4632: Fix ParentSelection regressions on...

2016-07-01 Thread PSUdaemon
Github user PSUdaemon closed the pull request at: https://github.com/apache/trafficserver/pull/781 --- 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 featu

[GitHub] trafficserver issue #781: TS-4632: Fix ParentSelection regressions on 6.2.x

2016-07-01 Thread PSUdaemon
Github user PSUdaemon commented on the issue: https://github.com/apache/trafficserver/pull/781 We can't merge this into 6.2.x. We need it in master, than I will cherry-pick it back. --- 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 #779: TS-4627: support TSRemapOSResponse in ts_lu...

2016-07-01 Thread jpeach
Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/779#discussion_r69363166 --- Diff: plugins/experimental/ts_lua/ts_lua.c --- @@ -116,6 +116,69 @@ TSRemapDeleteInstance(void *ih) return; } +void +TSRema

[GitHub] trafficserver pull request #779: TS-4627: support TSRemapOSResponse in ts_lu...

2016-07-01 Thread jpeach
Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/779#discussion_r69363133 --- Diff: plugins/experimental/ts_lua/ts_lua.c --- @@ -116,6 +116,69 @@ TSRemapDeleteInstance(void *ih) return; } +void +TSRema

[GitHub] trafficserver pull request #782: TS-4633: Fix an erroneous warning message w...

2016-07-01 Thread jrushford
Github user jrushford closed the pull request at: https://github.com/apache/trafficserver/pull/782 --- 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 featu

[GitHub] trafficserver pull request #779: TS-4627: support TSRemapOSResponse in ts_lu...

2016-07-01 Thread jpeach
Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/779#discussion_r69362522 --- Diff: plugins/experimental/ts_lua/ts_lua_http.c --- @@ -29,9 +29,33 @@ typedef enum { TS_LUA_CACHE_LOOKUP_SKIPPED } TSLuaCacheLookupResul

[GitHub] trafficserver pull request #779: TS-4627: support TSRemapOSResponse in ts_lu...

2016-07-01 Thread jpeach
Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/779#discussion_r69362504 --- Diff: plugins/experimental/ts_lua/ts_lua_http.c --- @@ -29,9 +29,33 @@ typedef enum { TS_LUA_CACHE_LOOKUP_SKIPPED } TSLuaCacheLookupResul

[GitHub] trafficserver issue #782: TS-4633: Fix an erroneous warning message when rea...

2016-07-01 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/782 Oh I see, the 6.2.0 PR is in #781. Great. --- 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 featu

[GitHub] trafficserver issue #781: TS-4632: Fix ParentSelection regressions on 6.2.x

2016-07-01 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/781 :+1: --- 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 f

[GitHub] trafficserver pull request #784: Temporarily disable ParentProxy API tests.

2016-07-01 Thread jpeach
Github user jpeach closed the pull request at: https://github.com/apache/trafficserver/pull/784 --- 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 #782: TS-4633: Fix an erroneous warning message when rea...

2016-07-01 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/782 :+1: Lets back port this to 6.2.0 as well, since we have to re-roll because of the other fix. --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] trafficserver issue #784: Temporarily disable ParentProxy API tests.

2016-07-01 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/784 :+1: --- 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 f

[GitHub] trafficserver issue #784: Temporarily disable ParentProxy API tests.

2016-07-01 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/784 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/299/ for details. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] trafficserver pull request #776: TS-4553: Brotli plugin

2016-07-01 Thread caricaturecm
Github user caricaturecm commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/776#discussion_r69356811 --- Diff: plugins/experimental/brotli_transform/brotli_transform.cc --- @@ -0,0 +1,248 @@ +/** + Licensed to the Apache Software Foundatio

[GitHub] trafficserver issue #784: Temporarily disable ParentProxy API tests.

2016-07-01 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/784 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/404/ for details. --- If your project is set up for it, you can reply to this email and have your r

[GitHub] trafficserver issue #782: TS-4633: Fix an erroneous warning message when rea...

2016-07-01 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/782 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/298/ for details. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] trafficserver pull request #783: TS-3816 : Replace ptr_len_cmp with memcmp

2016-07-01 Thread jpeach
Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/783#discussion_r69356010 --- Diff: proxy/http/HttpTransact.cc --- @@ -762,8 +762,8 @@ HttpTransact::StartRemapRequest(State *s) const char syntxt[] = "synthetic.txt"

[GitHub] trafficserver pull request #783: TS-3816 : Replace ptr_len_cmp with memcmp

2016-07-01 Thread jpeach
Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/783#discussion_r69355867 --- Diff: proxy/http/HttpTransact.cc --- @@ -249,7 +249,7 @@ find_server_and_update_current_info(HttpTransact::State *s) int host_len; con

[GitHub] trafficserver pull request #783: TS-3816 : Replace ptr_len_cmp with memcmp

2016-07-01 Thread jpeach
Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/783#discussion_r69355758 --- Diff: proxy/StatPages.cc --- @@ -83,7 +83,7 @@ StatPagesManager::handle_http(Continuation *cont, HTTPHdr *header) host_len = unescapi

[GitHub] trafficserver pull request #784: Temporarily disable ParentProxy API tests.

2016-07-01 Thread jpeach
GitHub user jpeach opened a pull request: https://github.com/apache/trafficserver/pull/784 Temporarily disable ParentProxy API tests. You can merge this pull request into a Git repository by running: $ git pull https://github.com/jpeach/trafficserver fix/disable-proxy-tests A

[GitHub] trafficserver issue #784: Temporarily disable ParentProxy API tests.

2016-07-01 Thread jpeach
Github user jpeach commented on the issue: https://github.com/apache/trafficserver/pull/784 @zwoop --- 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 t

[GitHub] trafficserver issue #782: TS-4633: Fix an erroneous warning message when rea...

2016-07-01 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/782 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/403/ for details. --- If your project is set up for it, you can reply to this email and have your r

[GitHub] trafficserver pull request #783: TS-3816 : Replace ptr_len_cmp with memcmp

2016-07-01 Thread strotyl
GitHub user strotyl opened a pull request: https://github.com/apache/trafficserver/pull/783 TS-3816 : Replace ptr_len_cmp with memcmp You can merge this pull request into a Git repository by running: $ git pull https://github.com/strotyl/trafficserver TS-3816 Alternatively yo

[GitHub] trafficserver pull request #782: TS-4633: Fix an erroneous warning message w...

2016-07-01 Thread jrushford
GitHub user jrushford opened a pull request: https://github.com/apache/trafficserver/pull/782 TS-4633: Fix an erroneous warning message when reading the parent.con… An erroneous warning message is logged when at startup when reading the parent.config file: "WARNING: [Par

[GitHub] trafficserver issue #781: TS-4632: Fix ParentSelection regressions on 6.2.x

2016-07-01 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/781 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/402/ for details. --- If your project is set up for it, you can reply to this email and have your r

[GitHub] trafficserver issue #781: TS-4632: Fix ParentSelection regressions on 6.2.x

2016-07-01 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/781 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/297/ for details. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] trafficserver pull request #781: TS-4632: Fix ParentSelection regressions on...

2016-07-01 Thread jrushford
GitHub user jrushford opened a pull request: https://github.com/apache/trafficserver/pull/781 TS-4632: Fix ParentSelection regressions on 6.2.x Remove an ink_release_assert artifact in ParentSelection.cc that causes nextParent() to fail and fix a warning message about parent_retry w

[GitHub] trafficserver pull request #775: TS-4598 : Coverity Null-Check after deref i...

2016-07-01 Thread zwoop
Github user zwoop closed the pull request at: https://github.com/apache/trafficserver/pull/775 --- 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 i

[GitHub] trafficserver issue #769: TS 4593: Extend ip_allow.config to filter destinat...

2016-07-01 Thread kawaiirice
Github user kawaiirice commented on the issue: https://github.com/apache/trafficserver/pull/769 dest_ip filtering has been tested for blocking connections at remap. --- 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 issue #769: TS 4593: Extend ip_allow.config to filter destinat...

2016-07-01 Thread kawaiirice
Github user kawaiirice commented on the issue: https://github.com/apache/trafficserver/pull/769 @shinrich --- 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 issue #777: TS-4623 Adds a pre-commit script for git

2016-07-01 Thread jpeach
Github user jpeach commented on the issue: https://github.com/apache/trafficserver/pull/777 👍 --- 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 #777: TS-4623 Adds a pre-commit script for git

2016-07-01 Thread jpeach
Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/777#discussion_r69330732 --- Diff: tools/pre-commit --- @@ -0,0 +1,66 @@ +#! /usr/bin/env bash +# +# Simple pre-commit hook script, enforcing clang-format on our tre

[GitHub] trafficserver pull request #780: TS-4629: Fix use of invalid iterator in Ser...

2016-07-01 Thread dragon512
Github user dragon512 commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/780#discussion_r69330409 --- Diff: proxy/http/HttpSessionManager.cc --- @@ -55,9 +55,10 @@ ServerSessionPool::ServerSessionPool() : Continuation(new_ProxyMutex()), m_ip_po

[GitHub] trafficserver pull request #780: TS-4629: Fix use of invalid iterator in Ser...

2016-07-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/780 --- 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 #780: TS-4629: Fix use of invalid iterator in ServerSess...

2016-07-01 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/780 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/401/ for details. --- If your project is set up for it, you can reply to this email and have your r

[GitHub] trafficserver issue #780: TS-4629: Fix use of invalid iterator in ServerSess...

2016-07-01 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/780 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/400/ for details. --- If your project is set up for it, you can reply to this email and have your r

[GitHub] trafficserver issue #780: TS-4629: Fix use of invalid iterator in ServerSess...

2016-07-01 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/780 Linux build *failed*! See https://ci.trafficserver.apache.org/job/Github-Linux/296/ for details. --- If your project is set up for it, you can reply to this email and have your reply app

[GitHub] trafficserver pull request #780: TS-4629: Fix use of invalid iterator in Ser...

2016-07-01 Thread SolidWallOfCode
GitHub user SolidWallOfCode opened a pull request: https://github.com/apache/trafficserver/pull/780 TS-4629: Fix use of invalid iterator in ServerSessionPool::purge. Add postfix increment to IPHashTable::iterator. You can merge this pull request into a Git repository by running:

[GitHub] trafficserver issue #775: TS-4598 : Coverity Null-Check after deref in Netwo...

2016-07-01 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/775 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/295/ for details. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] trafficserver issue #775: TS-4598 : Coverity Null-Check after deref in Netwo...

2016-07-01 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/775 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/399/ for details. --- If your project is set up for it, you can reply to this email and have your r

[GitHub] trafficserver issue #775: TS-4598 : Coverity Null-Check after deref in Netwo...

2016-07-01 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/775 [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, or

[GitHub] trafficserver issue #775: TS-4598 : Coverity Null-Check after deref in Netwo...

2016-07-01 Thread strotyl
Github user strotyl commented on the issue: https://github.com/apache/trafficserver/pull/775 Will do. --- 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 #779: TS-4627: support TSRemapOSResponse in ts_lua

2016-07-01 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/779 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/294/ for details. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] trafficserver issue #779: TS-4627: support TSRemapOSResponse in ts_lua

2016-07-01 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/779 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/398/ for details. --- If your project is set up for it, you can reply to this email and have your r

[GitHub] trafficserver pull request #779: TS-4627: support TSRemapOSResponse in ts_lu...

2016-07-01 Thread shukitchan
GitHub user shukitchan opened a pull request: https://github.com/apache/trafficserver/pull/779 TS-4627: support TSRemapOSResponse in ts_lua @jpeach pls take a look. Thanks. I want to add enhancement to ts_lua so I can write script to take advantage of the TSRemapOSResponse(

[GitHub] trafficserver pull request #772: TS-3726: Expose build_error_response

2016-07-01 Thread alhonen
Github user alhonen commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/772#discussion_r69261710 --- Diff: proxy/api/ts/ts.h --- @@ -2336,6 +2336,35 @@ tsapi uint64_t TSHttpTxnIdGet(TSHttpTxn txnp); tsapi TSReturnCode TSBase64Decode(const char