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);
