Skip to content

Commit

Permalink
http: initializes a field of ConnectionManagerImpl::ActiveStream::Sta…
Browse files Browse the repository at this point in the history
…te. (envoyproxy#36642)

While debugging an issue, it became clear that this field is not
properly initialized.

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:

---------

Signed-off-by: Biren Roy <birenroy@google.com>
  • Loading branch information
birenroy authored Oct 18, 2024
1 parent 803aea3 commit ee61634
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion source/common/http/conn_manager_impl.h
Original file line number Diff line number Diff line change
Expand Up @@ -338,7 +338,8 @@ class ConnectionManagerImpl : Logger::Loggable<Logger::Id::http>,
: codec_saw_local_complete_(false), codec_encode_complete_(false),
on_reset_stream_called_(false), is_zombie_stream_(false), successful_upgrade_(false),
is_internally_destroyed_(false), is_internally_created_(false), is_tunneling_(false),
decorated_propagate_(true), deferred_to_next_io_iteration_(false) {}
decorated_propagate_(true), deferred_to_next_io_iteration_(false),
deferred_end_stream_(false) {}

// It's possibly for the codec to see the completed response but not fully
// encode it.
Expand Down

0 comments on commit ee61634

Please sign in to comment.