Remove logging of header name and value length for headers with too long value. Remove LOG_FIRST_N statements added at cl/296224216. gfe-relnote: remove two LOG_FIRST_N statements. PiperOrigin-RevId: 297654216 Change-Id: Ib3f4df9495e49cbcb9acba9dcf3e614cdf3d710b
diff --git a/http2/hpack/decoder/hpack_decoder.cc b/http2/hpack/decoder/hpack_decoder.cc index bca83db..1013292 100644 --- a/http2/hpack/decoder/hpack_decoder.cc +++ b/http2/hpack/decoder/hpack_decoder.cc
@@ -66,10 +66,6 @@ HTTP2_CODE_COUNT_N(decompress_failure_3, 4, 23); return false; } else if (DetectError()) { - if (error_ == HpackDecodingError::kValueTooLong) { - HTTP2_LOG_FIRST_N(ERROR, 1) << "Header value too long for header name: " - << entry_buffer_.name().str(); - } HTTP2_CODE_COUNT_N(decompress_failure_3, 5, 23); return false; }
diff --git a/http2/hpack/decoder/hpack_whole_entry_buffer.cc b/http2/hpack/decoder/hpack_whole_entry_buffer.cc index de26c0d..da5613e 100644 --- a/http2/hpack/decoder/hpack_whole_entry_buffer.cc +++ b/http2/hpack/decoder/hpack_whole_entry_buffer.cc
@@ -94,9 +94,6 @@ HTTP2_DVLOG(1) << "Value length (" << len << ") is longer than permitted (" << max_string_size_bytes_ << ")"; - HTTP2_LOG_FIRST_N(ERROR, 1) - << "Header value length " << len << " exceeds limit of " - << max_string_size_bytes_; ReportError(HpackDecodingError::kValueTooLong); HTTP2_CODE_COUNT_N(decompress_failure_3, 21, 23); return;
diff --git a/http2/hpack/decoder/hpack_whole_entry_buffer.h b/http2/hpack/decoder/hpack_whole_entry_buffer.h index 27eb718..5c0bfe2 100644 --- a/http2/hpack/decoder/hpack_whole_entry_buffer.h +++ b/http2/hpack/decoder/hpack_whole_entry_buffer.h
@@ -79,8 +79,6 @@ void OnValueEnd() override; void OnDynamicTableSizeUpdate(size_t size) override; - const HpackDecoderStringBuffer& name() const { return name_; } - private: void ReportError(HpackDecodingError error);