Skip to content

Commit da39efa

Browse files
committed
Fix AuTest failure on Linux
1 parent eff9d49 commit da39efa

2 files changed

Lines changed: 19 additions & 6 deletions

File tree

src/proxy/http/HttpSM.cc

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4688,8 +4688,15 @@ HttpSM::track_connect_fail() const
46884688
// Excludes two cases:
46894689
// - Reused keep-alive connection: there is a known race between ATS reusing and the origin closing it.
46904690
// - Multiplexed origins (HTTP/2): stream-level failure does not indicate a connection failure.
4691-
if (policy >= 4 && !origin_multiplexed() && server_response_hdr_bytes == 0 && server_txn->is_first_transaction()) {
4692-
return true;
4691+
if (policy >= 4) {
4692+
bool multiplexed = false;
4693+
auto ssn = server_txn->get_proxy_ssn();
4694+
if (ssn != nullptr) {
4695+
multiplexed = static_cast<PoolableSession *>(ssn)->is_multiplexing();
4696+
}
4697+
if (!multiplexed && server_txn->is_first_transaction() && server_response_hdr_bytes == 0) {
4698+
return true;
4699+
}
46934700
}
46944701

46954702
return false;

tests/gold_tests/connect_down_policy/replay/connect_down_policy_4.replay.yaml

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,19 +26,19 @@ autest:
2626
description: 'Verify connect down policy 4 - origin closes connection without sending a response'
2727

2828
dns:
29-
name: 'dns-dp-4'
29+
name: 'dns-policy-4'
3030
records:
3131
backend1.example.com: ["127.0.0.1"]
3232
backend2.example.com: ["127.0.0.1"]
3333

3434
server:
35-
name: 'server-dp-4'
35+
name: 'server-policy-4'
3636

3737
client:
38-
name: 'client-dp-4'
38+
name: 'client-policy-4'
3939

4040
ats:
41-
name: 'ts-dp-4'
41+
name: 'ts-policy-4'
4242
process_config:
4343
enable_cache: false
4444

@@ -93,6 +93,9 @@ sessions:
9393
proxy-request:
9494
expect: absent
9595

96+
server-response:
97+
status: 200
98+
9699
proxy-response:
97100
status: 500
98101

@@ -125,5 +128,8 @@ sessions:
125128
proxy-request:
126129
expect: absent
127130

131+
server-response:
132+
status: 200
133+
128134
proxy-response:
129135
status: 500

0 commit comments

Comments
 (0)