[trafficserver] branch 9.0.x updated: Fixes a bug where the nexthop markNextHop method to mark a host down is not called when because the wrapper function was not used.

[zwoop]

This is an automated email from the ASF dual-hosted git repository.

zwoop pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/9.0.x by this push:
     new 22b877b  Fixes a bug where the nexthop markNextHop method to mark a 
host down is not called when because the wrapper function was not used.
22b877b is described below

commit 22b877b201e72d0b40f5df7d4268a59078d55bb7
Author: John Rushford <jrushf...@apache.org>
AuthorDate: Wed Apr 8 16:38:52 2020 +0000

    Fixes a bug where the nexthop markNextHop method
    to mark a host down is not called when because
    the wrapper function was not used.
    
    (cherry picked from commit 90357576c4cbd8c172047955d385b7ba0cd62627)
---
 proxy/http/HttpTransact.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/proxy/http/HttpTransact.cc b/proxy/http/HttpTransact.cc
index cafab42..79d87fe 100644
--- a/proxy/http/HttpTransact.cc
+++ b/proxy/http/HttpTransact.cc
@@ -3603,7 +3603,7 @@ HttpTransact::handle_response_from_parent(State *s)
         //  us to mark the parent down
         if (s->current.state == CONNECTION_ERROR) {
           HTTP_INCREMENT_DYN_STAT(http_total_parent_marked_down_count);
-          s->parent_params->markParentDown(&s->parent_result, 
s->txn_conf->parent_fail_threshold, s->txn_conf->parent_retry_time);
+          markParentDown(s);
         }
         // We are done so look for another parent if any
         next_lookup = find_server_and_update_current_info(s);
@@ -3615,7 +3615,7 @@ HttpTransact::handle_response_from_parent(State *s)
       TxnDebug("http_trans", "[handle_response_from_parent] Error. No more 
retries.");
       if (s->current.state == CONNECTION_ERROR) {
         HTTP_INCREMENT_DYN_STAT(http_total_parent_marked_down_count);
-        s->parent_params->markParentDown(&s->parent_result, 
s->txn_conf->parent_fail_threshold, s->txn_conf->parent_retry_time);
+        markParentDown(s);
       }
       s->parent_result.result = PARENT_FAIL;
       next_lookup             = HOST_NONE;