1
87%
master: 89%

Ran 18 Mar 2021 11:13PM UTC

Files 430

Run time 1min

Badge

Committed 18 Mar 2021 11:00PM UTC coverage: 87.075% (-0.001%) from 87.076%

Job # GRADLE_OPTS=-Xmx512m LDFLAGS=-L/tmp/protobuf/lib CXXFLAGS=-I/tmp/protobuf/include LD_LIBRARY_PATH=/tmp/protobuf/lib

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

25890 of 29733 relevant lines covered (87.07%)

0.87 hits per line

grpc / grpc-java / 21691 / 1
87%
master: 89%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 21691.1 (GRADLE_OPTS=-Xmx512m LDFLAGS=-L/tmp/protobuf/lib CXXFLAGS=-I/tmp/protobuf/include LD_LIBRARY_PATH=/tmp/protobuf/lib)

grpc / grpc-java / 21691 / 1 87% master: 89%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 21691.1 (GRADLE_OPTS=-Xmx512m LDFLAGS=-L/tmp/protobuf/lib CXXFLAGS=-I/tmp/protobuf/include LD_LIBRARY_PATH=/tmp/protobuf/lib)

grpc / grpc-java / 21691 / 1
87%
master: 89%

README BADGES
x