[GitHub] trafficserver pull request: TS-3779: Add per-host error pages to b...

2015-07-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/257 --- 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-3780: Custom log add incoming inter...

2015-07-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/258 --- 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-3774: Fix memory leak issue in host...

2015-07-22 Thread jacksontj
Github user jacksontj commented on the pull request: https://github.com/apache/trafficserver/pull/260#issuecomment-123926596 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

[GitHub] trafficserver pull request: Test for empty resolv_conf

2015-07-22 Thread jacksontj
GitHub user jacksontj opened a pull request: https://github.com/apache/trafficserver/pull/261 Test for empty resolv_conf Test case for TS-3792 You can merge this pull request into a Git repository by running: $ git pull https://github.com/jacksontj/trafficserver master Alterna

[GitHub] trafficserver pull request: TS-3780: Custom log add incoming inter...

2015-07-22 Thread ericcarlschwartz
Github user ericcarlschwartz commented on the pull request: https://github.com/apache/trafficserver/pull/258#issuecomment-123753341 Sounds good to me just wanted to make sure they both get closed out! --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] trafficserver pull request: TS-3781: Add the log field "pqsp" (ser...

2015-07-22 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/255#issuecomment-123612759 @bgaff Please review and commit. Also, make sure the mnemonics for this new tag (and TS-3780) are in line with existing standards. --- If your project is set up f

[GitHub] trafficserver pull request: TS-3780: Custom log add incoming inter...

2015-07-22 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/258#issuecomment-123593668 Yeah, close TS-2152 as a dupe of TS-3780, since there's so much work done on this PR already. In general, please file Jira's before work / pull requests are done, s

[GitHub] trafficserver pull request: TS-3780: Custom log add incoming inter...

2015-07-22 Thread bgaff
Github user bgaff commented on the pull request: https://github.com/apache/trafficserver/pull/258#issuecomment-123589377 Hi @ericcarlschwartz , if you're cool with he proposed patch I'll land this one and we can close out TS-2152 as a dupe. --- If your project is set up for it, you c