remove unused code

Author: traeak

include/proxy/http/remap/NextHopSelectionStrategy.h

@@ -198,8 +198,6 @@ class NextHopSelectionStrategy
                            const time_t now = 0);
   bool         nextHopExists(TSHttpTxn txnp, void *ih = nullptr);
 
-  void setHostHeader(TSHttpTxn txnp, const char *hostname);
-
   virtual ParentRetry_t responseIsRetryable(int64_t sm_id, HttpTransact::CurrentInfo &current_info, HTTPStatus response_code);
 
   void retryComplete(TSHttpTxn txn, const char *hostname, const int port);

src/proxy/http/HttpTransact.cc

@@ -701,6 +701,7 @@ find_server_and_update_current_info(HttpTransact::State *s)
 
   if (s->parent_result.result == ParentResultType::SPECIFIED && host_override(s)) {
     char const *const hostname = s->parent_result.hostname;
+    TxnDbg(dbg_ctl_http_trans, "overriding host header with parent %s", hostname);
     if (!s->hdr_info.server_request.valid()) {
       s->hdr_info.client_request.value_set(static_cast<std::string_view>(MIME_FIELD_HOST), hostname);
       s->hdr_info.client_request.mark_target_dirty();

src/proxy/http/remap/NextHopConsistentHash.cc

@@ -547,8 +547,4 @@ NextHopConsistentHash::findNextHop(TSHttpTxn txnp, void * /* ih ATS_UNUSED */, t
     NH_Dbg(NH_DBG_CTL, "[%" PRIu64 "] result.result: %s set hostname null port 0 retry false", sm_id,
            ParentResultStr[static_cast<int>(result.result)]);
   }
-
-  setHostHeader(txnp, result.hostname);
-
-  return;
 }

src/proxy/http/remap/NextHopRoundRobin.cc

@@ -170,7 +170,6 @@ NextHopRoundRobin::findNextHop(TSHttpTxn txnp, void * /* ih ATS_UNUSED */, time_
       result->last_parent = cur_hst_index;
       result->last_group  = cur_grp_index;
       result->retry       = parentRetry;
-      setHostHeader(txnp, result->hostname);
       ink_assert(result->hostname != nullptr);
       ink_assert(result->port != 0);
       NH_Dbg(NH_DBG_CTL, "[%" PRIu64 "] Chosen parent = %s.%d", sm_id, result->hostname, result->port);

src/proxy/http/remap/NextHopSelectionStrategy.cc

@@ -265,22 +265,6 @@ NextHopSelectionStrategy::markNextHop(TSHttpTxn txnp, const char *hostname, cons
   return passive_health.markNextHop(txnp, hostname, port, status, ih, now);
 }
 
-void
-NextHopSelectionStrategy::setHostHeader(TSHttpTxn txnp, const char *hostname)
-{
-  if (host_override && nullptr != hostname) {
-    HttpSM *const sm = reinterpret_cast<HttpSM *>(txnp);
-    if (!sm->t_state.hdr_info.server_request.valid()) {
-      sm->t_state.hdr_info.client_request.value_set(static_cast<std::string_view>(MIME_FIELD_HOST), hostname);
-      sm->t_state.hdr_info.client_request.mark_target_dirty();
-    } else {
-      sm->t_state.hdr_info.server_request.value_set(static_cast<std::string_view>(MIME_FIELD_HOST), hostname);
-      sm->t_state.hdr_info.server_request.mark_target_dirty();
-    }
-    NH_Dbg(NH_DBG_CTL, "[%" PRIu64 "] overriding host header with parent %s", sm->sm_id, hostname);
-  }
-}
-
 bool
 NextHopSelectionStrategy::nextHopExists(TSHttpTxn txnp, void * /* ih ATS_UNUSED */)
 {