Skip to content

Commit

Permalink
quic: migrates one more reference to quiche::HttpHeaderBlock (envoypr…
Browse files Browse the repository at this point in the history
…oxy#35889)

Commit Message: quic: migrates one more reference to
quiche::HttpHeaderBlock
Additional Description:
Risk Level: none
Testing: ran unit and integration tests locally
Docs Changes:
Release Notes:
Platform Specific Features:

Signed-off-by: Biren Roy <[email protected]>
  • Loading branch information
birenroy authored Aug 29, 2024
1 parent db12a70 commit dccb77a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions source/common/quic/envoy_quic_stream.cc
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ void EnvoyQuicStream::encodeData(Buffer::Instance& data, bool end_stream) {
}
}

void EnvoyQuicStream::encodeTrailersImpl(spdy::Http2HeaderBlock&& trailers) {
void EnvoyQuicStream::encodeTrailersImpl(quiche::HttpHeaderBlock&& trailers) {
if (quic_stream_.write_side_closed()) {
IS_ENVOY_BUG("encodeTrailers is called on write-closed stream.");
return;
Expand Down Expand Up @@ -126,7 +126,7 @@ void serializeMetadata(const Http::MetadataMapPtr& metadata, quic::QuicStreamId
quic::QpackEncoder qpack_encoder(&decoder_stream_error_delegate, quic::HuffmanEncoding::kDisabled,
quic::CookieCrumbling::kDisabled);

spdy::Http2HeaderBlock header_block;
quiche::HttpHeaderBlock header_block;
for (const auto& [key, value] : *metadata) {
header_block.AppendValueOrAddHeader(key, value);
}
Expand Down
2 changes: 1 addition & 1 deletion source/common/quic/envoy_quic_stream.h
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,7 @@ class EnvoyQuicStream : public virtual Http::StreamEncoder,

StreamInfo::BytesMeterSharedPtr& mutableBytesMeter() { return bytes_meter_; }

void encodeTrailersImpl(spdy::Http2HeaderBlock&& trailers);
void encodeTrailersImpl(quiche::HttpHeaderBlock&& trailers);

// Converts `header_list` into a new `Http::MetadataMap`.
std::unique_ptr<Http::MetadataMap>
Expand Down

0 comments on commit dccb77a

Please sign in to comment.