21652
89%

Ran 16 Mar 2021 11:49PM UTC

Jobs 1

Files 434

Run time 1min

Badge

Embed ▾

pending completion

Build # 21652

Build Type

push

travis-ci

Committed by

ejona86

Commit Message

netty: Add Http2Headers.setLong() for inbound headers

Starting in Netty 4.1.60, Netty will validate Content-Length headers
using getAll() and setLong(). While getAll() was documented as only used
in tests, it doesn't appear it was currently used in any tests.

While Http2NettyTest.contentLengthPermitted() was added to confirm that
Content-Length works, it won't actually exercise any interesting
behavior until we upgrade to Netty 4.1.60. However, I did test with
Netty 4.1.60 and it reproduced the failure in
https://github.com/grpc/grpc-java/issues/7953 and passed with this
change.

Since Netty is now observing/modifying the headers, it would seem
appropriate to implement a substantial portion of the Http2Headers API.
However, the surface is much larger than we'd want to implement for a
'quick fix' that could be backported. In addition, it seems much of the
API is just convenience methods, so it is probably appropriate to split
out a AbstractHeaders class from DefaultHeaders in Netty that doesn't
make any assumptions about the header storage mechanism.

Run Details

26383 of 30067 relevant lines covered (87.75%)

0.88 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	21652.1 (GRADLE_OPTS=-Xmx512m LDFLAGS=-L/tmp/protobuf/lib CXXFLAGS=-I/tmp/protobuf/include LD_LIBRARY_PATH=/tmp/protobuf/lib)	16 Mar 2021 11:49PM UTC	0	87.75	Travis Job 21652.1

Source Files on build 21652

Detailed source file information is not available for this build.

grpc / grpc-java / 21652
89%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 21652

grpc / grpc-java / 21652 89%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 21652

grpc / grpc-java / 21652
89%

README BADGES
x