[trafficserver] branch master updated: Adding host URL to NextHop Strategies debug message during host mark down. (#9960)

[eze]

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

eze pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new 930236bef9 Adding host URL to NextHop Strategies debug message during 
host mark down. (#9960)
930236bef9 is described below

commit 930236bef942b73520eacd0f2a9b9b9038ec643f
Author: Tyler Morgan <tyler_mor...@cable.comcast.com>
AuthorDate: Mon Jul 24 15:42:10 2023 -0400

    Adding host URL to NextHop Strategies debug message during host mark down. 
(#9960)
    
    * adds feature to close #8050
    closes #8050
---
 proxy/http/remap/NextHopHealthStatus.cc | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/proxy/http/remap/NextHopHealthStatus.cc 
b/proxy/http/remap/NextHopHealthStatus.cc
index b553b95104..7cff756187 100644
--- a/proxy/http/remap/NextHopHealthStatus.cc
+++ b/proxy/http/remap/NextHopHealthStatus.cc
@@ -76,6 +76,7 @@ NextHopHealthStatus::markNextHop(TSHttpTxn txn, const char 
*hostname, const int
   int64_t fail_threshold  = sm->t_state.txn_conf->parent_fail_threshold;
   int64_t retry_time      = sm->t_state.txn_conf->parent_retry_time;
   uint32_t new_fail_count = 0;
+  const char *host        = sm->t_state.request_data.get_host();
 
   // make sure we're called back with a result structure for a parent
   // that is being retried.
@@ -123,7 +124,9 @@ NextHopHealthStatus::markNextHop(TSHttpTxn txn, const char 
*hostname, const int
           new_fail_count = h->failCount += 1;
         }
       } // end lock guard
-      NH_Note("[%" PRId64 "] NextHop %s marked as down %s", sm_id, 
(result.retry) ? "retry" : "initially", h->hostname.c_str());
+
+      NH_Note("[%" PRId64 "] NextHop %s marked as down %s", sm_id, 
(result.retry) ? "retry" : "initially", host ? host : "(null)");
+
     } else {
       // if the last failure was outside the retry window, set the failcount 
to 1 and failedAt to now.
       { // lock guard