Skip to content

Commit

Permalink
Post merge fix
Browse files Browse the repository at this point in the history
Signed-off-by: Yan Avlasov <yavlasov@google.com>
  • Loading branch information
yanavlasov committed Jul 30, 2024
1 parent 01ae68b commit 1f269a4
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions source/common/quic/envoy_quic_utils.cc
Original file line number Diff line number Diff line change
Expand Up @@ -90,6 +90,8 @@ quic::QuicRstStreamErrorCode envoyResetReasonToQuicRstError(Http::StreamResetRea
case Http::StreamResetReason::RemoteRefusedStreamReset:
case Http::StreamResetReason::RemoteReset:
IS_ENVOY_BUG("Remote reset shouldn't be initiated by self.");
case Http::StreamResetReason::Http1PrematureUpstreamHalfClose:
IS_ENVOY_BUG("H/1 premature response reset is not applicable to H/3.");
}

ENVOY_LOG_MISC(error, absl::StrCat("Unknown reset reason: ", reason));
Expand Down
2 changes: 1 addition & 1 deletion test/common/router/router_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -3476,7 +3476,7 @@ TEST_F(RouterTest, NoRetryWithBodyLimitWithUpstreamHalfCloseEnabled) {
Buffer::OwnedImpl body("t");
router_->decodeData(body, false);
EXPECT_EQ(1U,
callbacks_.route_->route_entry_.virtual_cluster_.stats().upstream_rq_total_.value());
callbacks_.route_->virtual_host_.virtual_cluster_.stats().upstream_rq_total_.value());

Http::ResponseHeaderMapPtr response_headers(
new Http::TestResponseHeaderMapImpl{{":status", "200"}});
Expand Down

0 comments on commit 1f269a4

Please sign in to comment.