Skip to content

Commit

Permalink
stream-info: Remove runtime guard `copy_response_code_to_downstream_s…
Browse files Browse the repository at this point in the history
…tream_info` (envoyproxy#33639)

Signed-off-by: Jacob Bohanon <jacob.bohanon@solo.io>
  • Loading branch information
jbohanon authored Apr 25, 2024
1 parent 21b406b commit 835ed26
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 5 deletions.
3 changes: 3 additions & 0 deletions changelogs/current.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,9 @@ removed_config_or_runtime:
- area: http
change: |
Removed ``envoy.reloadable_features.lowercase_scheme`` runtime flag and lagacy code paths.
- area: router
change: |
Removed ``envoy.reloadable_features.copy_response_code_to_downstream_stream_info`` runtime flag and legacy code paths.
new_features:
- area: matching
Expand Down
5 changes: 1 addition & 4 deletions source/common/router/router.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1655,10 +1655,7 @@ void Filter::onUpstreamHeaders(uint64_t response_code, Http::ResponseHeaderMapPt
callbacks_->streamInfo().setResponseCodeDetails(
StreamInfo::ResponseCodeDetails::get().ViaUpstream);

if (Runtime::runtimeFeatureEnabled(
"envoy.reloadable_features.copy_response_code_to_downstream_stream_info")) {
callbacks_->streamInfo().setResponseCode(response_code);
}
callbacks_->streamInfo().setResponseCode(response_code);

// TODO(zuercher): If access to response_headers_to_add (at any level) is ever needed outside
// Router::Filter we'll need to find a better location for this work. One possibility is to
Expand Down
1 change: 0 additions & 1 deletion source/common/runtime/runtime_features.cc
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,6 @@ RUNTIME_GUARD(envoy_reloadable_features_avoid_zombie_streams);
RUNTIME_GUARD(envoy_reloadable_features_check_mep_on_first_eject);
RUNTIME_GUARD(envoy_reloadable_features_conn_pool_delete_when_idle);
RUNTIME_GUARD(envoy_reloadable_features_convert_legacy_lb_config);
RUNTIME_GUARD(envoy_reloadable_features_copy_response_code_to_downstream_stream_info);
RUNTIME_GUARD(envoy_reloadable_features_defer_processing_backedup_streams);
RUNTIME_GUARD(envoy_reloadable_features_dfp_mixed_scheme);
RUNTIME_GUARD(envoy_reloadable_features_disallow_quic_client_udp_mmsg);
Expand Down

0 comments on commit 835ed26

Please sign in to comment.