You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2022/07/26 22:30:30 UTC
[trafficserver] branch 9.2.x updated: Fix reverting PR#7302 (#8975)
This is an automated email from the ASF dual-hosted git repository.
zwoop pushed a commit to branch 9.2.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/9.2.x by this push:
new 9348455ba Fix reverting PR#7302 (#8975)
9348455ba is described below
commit 9348455ba6bcd29512a8839cdba9aa40b799160f
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Fri Jul 22 07:29:20 2022 +0900
Fix reverting PR#7302 (#8975)
PR#7302 was reverted by PR#8316 as an incompatible change for 9.2.0.
It looks like the revert commit has a mistake that made a crash by
calling `HttpSM::send_origin_throttled_response()` twice.
(cherry picked from commit 3cccd2d94d73fe9aa8452adaa9fdc30ad9215e84)
---
proxy/http/HttpSM.cc | 1 -
1 file changed, 1 deletion(-)
diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index e1bd524c5..84a7ccf50 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -5313,7 +5313,6 @@ HttpSM::do_http_server_open(bool raw)
ct_state.Warn_Blocked(&t_state.txn_conf->outbound_conntrack, sm_id, ccount - 1, &t_state.current.server->dst_addr.sa,
debug_on && is_debug_tag_set("http") ? "http" : nullptr);
- send_origin_throttled_response();
return;
} else {