[GitHub] trafficserver pull request #786: TS-4396: fix number_of_redirections off-by-...

2016-07-06 Thread mingzym
Github user mingzym commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/786#discussion_r69765684 --- Diff: proxy/http/HttpSM.cc --- @@ -1954,9 +1954,7 @@ HttpSM::state_read_server_response_header(int event, void *data

[GitHub] trafficserver pull request #786: TS-4396: fix number_of_redirections off-by-...

2016-07-06 Thread mingzym
Github user mingzym commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/786#discussion_r69764301 --- Diff: proxy/http/HttpSM.cc --- @@ -7618,6 +7616,8 @@ HttpSM::do_redirect() return; } + redirection_tries

[GitHub] trafficserver pull request #786: TS-4396: fix number_of_redirections off-by-...

2016-07-02 Thread mingzym
GitHub user mingzym opened a pull request: https://github.com/apache/trafficserver/pull/786 TS-4396: fix number_of_redirections off-by-one please test You can merge this pull request into a Git repository by running: $ git pull https://github.com/mingzym/trafficserver TS-4396

[GitHub] trafficserver issue #593: TS-4156: remove the traffic_sac, stand alone log c...

2016-06-28 Thread mingzym
Github user mingzym commented on the issue: https://github.com/apache/trafficserver/pull/593 @masaori335 here we go :D --- 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

[GitHub] trafficserver pull request #593: TS-4156: remove the traffic_sac, stand alon...

2016-06-28 Thread mingzym
Github user mingzym closed the pull request at: https://github.com/apache/trafficserver/pull/593 --- 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: Prevent LogFile from calling NULL refe...

2016-04-21 Thread mingzym
Github user mingzym closed the pull request at: https://github.com/apache/trafficserver/pull/591 --- 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: Remove duplicate log at HttpSM::do_hos...

2016-04-21 Thread mingzym
Github user mingzym closed the pull request at: https://github.com/apache/trafficserver/pull/557 --- 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: TS-4156: remove the traffic_sac, stand...

2016-04-21 Thread mingzym
GitHub user mingzym opened a pull request: https://github.com/apache/trafficserver/pull/593 TS-4156: remove the traffic_sac, stand alone log collation server TS-4156: remove the traffic_sac, stand alone log collation server https://issues.apache.org/jira/browse/TS-4156 You