[GitHub] trafficserver pull request: add redirection test

2015-04-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/183 --- 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: add redirection test

2015-04-22 Thread ffcai
Github user ffcai commented on the pull request: https://github.com/apache/trafficserver/pull/183#issuecomment-95257442 Hi @jacksontj , I find that it's not related to httpbin. I happen to find a trivial bug which would cause redirection fails. I send out another pull request (htt

[GitHub] trafficserver pull request: add redirection test

2015-04-16 Thread jacksontj
Github user jacksontj commented on the pull request: https://github.com/apache/trafficserver/pull/183#issuecomment-93805512 Can you try applying 5a367b6 to your tsqa? I think the wsgiserver I'm using right now only supports a single connection-- this should make it multithreaded and h

[GitHub] trafficserver pull request: add redirection test

2015-04-16 Thread ffcai
Github user ffcai commented on the pull request: https://github.com/apache/trafficserver/pull/183#issuecomment-93790194 I just find that the local httpbin (start with like `gunicorn httpbin:app`) acts different with http://httpbin.org. And, this test case fails for local httpbin :( .

[GitHub] trafficserver pull request: add redirection test

2015-04-14 Thread ffcai
Github user ffcai commented on the pull request: https://github.com/apache/trafficserver/pull/183#issuecomment-92697189 Thanks @jacksontj I'll really glad that you add it to the framework. I'll update my tests as soon. --- If your project is set up for it, you can reply to this email

[GitHub] trafficserver pull request: add redirection test

2015-04-10 Thread jacksontj
Github user jacksontj commented on the pull request: https://github.com/apache/trafficserver/pull/183#issuecomment-91744591 @ffcai Do you mind updating your tests to use the in-build httpbin? If not I can do it, but I'd rather you get the karma for the awesome find :) --- If your pro

[GitHub] trafficserver pull request: add redirection test

2015-04-07 Thread jacksontj
Github user jacksontj commented on the pull request: https://github.com/apache/trafficserver/pull/183#issuecomment-90764490 I've added the wrapper stuff (for tracking) to tqsa (https://git-wip-us.apache.org/repos/asf?p=trafficserver-qa.git;a=commit;h=9e99c096d7147b4253f3ba210b079d3ea26

[GitHub] trafficserver pull request: add redirection test

2015-04-07 Thread jacksontj
Github user jacksontj commented on the pull request: https://github.com/apache/trafficserver/pull/183#issuecomment-90752166 Ooo, that sounds awesome! The main reason for having the origins within the framework is so we can see request/response headers on the origin side (since we may

[GitHub] trafficserver pull request: add redirection test

2015-03-27 Thread ffcai
Github user ffcai commented on the pull request: https://github.com/apache/trafficserver/pull/183#issuecomment-86863891 Hi @jacksontj , I hope to discuss with you about [httpbin](http://httpbin.org/) before the merge. It's a HTTP Request & Response Service, implemented by pytho

[GitHub] trafficserver pull request: add redirection test

2015-03-27 Thread ffcai
GitHub user ffcai opened a pull request: https://github.com/apache/trafficserver/pull/183 add redirection test You can merge this pull request into a Git repository by running: $ git pull https://github.com/ffcai/trafficserver tests Alternatively you can review and apply thes