(trafficserver) branch master updated: Cleanup: HttpTransact::StateMachineAction_t (#12533)

[masaori]

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

masaori 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 6f7cf27458 Cleanup: HttpTransact::StateMachineAction_t (#12533)
6f7cf27458 is described below

commit 6f7cf27458b525aa6b76ce97c9db6d63ccf66311
Author: Masaori Koshiba <masa...@apache.org>
AuthorDate: Thu Oct 2 11:00:27 2025 +0900

    Cleanup: HttpTransact::StateMachineAction_t (#12533)
---
 include/proxy/http/HttpTransact.h |  2 --
 src/proxy/http/HttpDebugNames.cc  |  6 ------
 src/proxy/http/HttpSM.cc          | 20 --------------------
 3 files changed, 28 deletions(-)

diff --git a/include/proxy/http/HttpTransact.h 
b/include/proxy/http/HttpTransact.h
index 66e53a889d..34cb0ae0be 100644
--- a/include/proxy/http/HttpTransact.h
+++ b/include/proxy/http/HttpTransact.h
@@ -388,7 +388,6 @@ public:
 
     ORIGIN_SERVER_OPEN,
     ORIGIN_SERVER_RAW_OPEN,
-    ORIGIN_SERVER_RR_MARK_DOWN,
 
     READ_PUSH_HDR,
     STORE_PUSH_BODY,
@@ -407,7 +406,6 @@ public:
     SERVER_PARSE_NEXT_HDR,
     TRANSFORM_READ,
     SSL_TUNNEL,
-    CONTINUE,
 
     API_SM_START,
     API_READ_REQUEST_HDR,
diff --git a/src/proxy/http/HttpDebugNames.cc b/src/proxy/http/HttpDebugNames.cc
index a6f83e587c..e06e50159f 100644
--- a/src/proxy/http/HttpDebugNames.cc
+++ b/src/proxy/http/HttpDebugNames.cc
@@ -398,9 +398,6 @@ 
HttpDebugNames::get_action_name(HttpTransact::StateMachineAction_t e)
   case HttpTransact::StateMachineAction_t::ORIGIN_SERVER_RAW_OPEN:
     return ("StateMachineAction_t::ORIGIN_SERVER_RAW_OPEN");
 
-  case HttpTransact::StateMachineAction_t::ORIGIN_SERVER_RR_MARK_DOWN:
-    return ("StateMachineAction_t::ORIGIN_SERVER_RR_MARK_DOWN");
-
   case HttpTransact::StateMachineAction_t::READ_PUSH_HDR:
     return ("StateMachineAction_t::READ_PUSH_HDR");
 
@@ -431,9 +428,6 @@ 
HttpDebugNames::get_action_name(HttpTransact::StateMachineAction_t e)
   case HttpTransact::StateMachineAction_t::SSL_TUNNEL:
     return ("StateMachineAction_t::SSL_TUNNEL");
 
-  case HttpTransact::StateMachineAction_t::CONTINUE:
-    return ("StateMachineAction_t::CONTINUE");
-
   case HttpTransact::StateMachineAction_t::API_READ_REQUEST_HDR:
     return ("StateMachineAction_t::API_READ_REQUEST_HDR");
 
diff --git a/src/proxy/http/HttpSM.cc b/src/proxy/http/HttpSM.cc
index 7eceaa7e32..994aaef6a7 100644
--- a/src/proxy/http/HttpSM.cc
+++ b/src/proxy/http/HttpSM.cc
@@ -8243,21 +8243,6 @@ HttpSM::set_next_state()
     break;
   }
 
-  case HttpTransact::StateMachineAction_t::ORIGIN_SERVER_RR_MARK_DOWN: {
-    HTTP_SM_SET_DEFAULT_HANDLER(&HttpSM::state_mark_os_down);
-    ATS_PROBE(next_state_SM_ACTION_ORIGIN_SERVER_RR_MARK_DOWN);
-
-    ink_assert(t_state.dns_info.looking_up == ResolveInfo::ORIGIN_SERVER);
-
-    // TODO: This might not be optimal (or perhaps even correct), but it will
-    // effectively mark the host as down. What's odd is that state_mark_os_down
-    // above isn't triggering.
-    HttpSM::do_hostdb_update_if_necessary();
-
-    do_hostdb_lookup();
-    break;
-  }
-
   case HttpTransact::StateMachineAction_t::SSL_TUNNEL: {
     t_state.api_next_action = 
HttpTransact::StateMachineAction_t::API_SEND_RESPONSE_HDR;
     do_api_callout();
@@ -8332,11 +8317,6 @@ HttpSM::set_next_state()
     break;
   }
 
-  case HttpTransact::StateMachineAction_t::CONTINUE: {
-    ink_release_assert(!"Not implemented");
-    break;
-  }
-
   default: {
     ink_release_assert(!"Unknown next action");
   }