[GitHub] trafficserver pull request: Correct docs for timing fields

2015-08-28 Thread jacksontj
Github user jacksontj closed the pull request at: https://github.com/apache/trafficserver/pull/285 --- 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: Correct docs for timing fields

2015-08-28 Thread jacksontj
Github user jacksontj commented on the pull request: https://github.com/apache/trafficserver/pull/285#issuecomment-135924707 Merged --- 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 featur

[GitHub] trafficserver pull request: Correct docs for timing fields

2015-08-28 Thread jacksontj
GitHub user jacksontj opened a pull request: https://github.com/apache/trafficserver/pull/285 Correct docs for timing fields You can merge this pull request into a Git repository by running: $ git pull https://github.com/jacksontj/trafficserver master Alternatively you can re

[GitHub] trafficserver pull request: TS-3874: Header-rewrite: support multi...

2015-08-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/284 --- 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: TS-3874: Header-rewrite: support multi...

2015-08-28 Thread sudheerv
Github user sudheerv commented on the pull request: https://github.com/apache/trafficserver/pull/284#issuecomment-135915883 lgtm --- 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 e

[GitHub] trafficserver pull request: TS-3740: header_rewrite set-redirect e...

2015-08-28 Thread sudheerv
Github user sudheerv commented on the pull request: https://github.com/apache/trafficserver/pull/253#issuecomment-135914324 Patch applied to master. --- 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

[GitHub] trafficserver pull request: TS-3874: Header-rewrite: support multi...

2015-08-28 Thread jpeach
Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/284#discussion_r38252920 --- Diff: plugins/header_rewrite/conditions.cc --- @@ -237,13 +236,22 @@ ConditionHeader::append_value(std::string &s, const Resources &res) }

[GitHub] trafficserver pull request: TS-3874: Header-rewrite: support multi...

2015-08-28 Thread jpeach
Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/284#discussion_r38252894 --- Diff: doc/reference/plugins/header_rewrite.en.rst --- @@ -208,3 +208,44 @@ Examples rm-header Set-Cookie counter plugin.header_rewrite.

[GitHub] trafficserver pull request: TS-3874: Header-rewrite: support multi...

2015-08-28 Thread jpeach
Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/284#discussion_r38252875 --- Diff: doc/reference/plugins/header_rewrite.en.rst --- @@ -208,3 +208,44 @@ Examples rm-header Set-Cookie counter plugin.header_rewrite.

[GitHub] trafficserver pull request: TS-3874: Header-rewrite: support multi...

2015-08-28 Thread jacksontj
GitHub user jacksontj opened a pull request: https://github.com/apache/trafficserver/pull/284 TS-3874: Header-rewrite: support multiple header values in conditionals You can merge this pull request into a Git repository by running: $ git pull https://github.com/jacksontj/traff

[GitHub] trafficserver pull request: Test for empty resolv_conf

2015-08-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/261 --- 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

Re: [GitHub] trafficserver pull request: This is the fix for TS-3848.

2015-08-28 Thread Pushkar Pradhan
Hello,Can someone review this PR?Thanks, pushkar On Thursday, August 27, 2015 11:53 AM, pushkarpradhan wrote: GitHub user pushkarpradhan opened a pull request:     https://github.com/apache/trafficserver/pull/282     This is the fix for TS-3848.     The values for wait_for_cache