21691
87%
master: 89%

Ran 18 Mar 2021 11:13PM UTC

Jobs 1

Files 430

Run time 1min

Badge

Embed ▾

pending completion

Build # 21691

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

Uncovered Existing Lines

Lines	Coverage	∆	File
1	97.3	-1.35%	../context/src/main/java/io/grpc/Deadline.java
2	80.6	-0.86%	../core/src/main/java/io/grpc/internal/DelayedClientCall.java
2	90.63	-6.25%	../netty/src/main/java/io/grpc/netty/ClientTransportLifecycleManager.java
2	73.2	-2.06%	../okhttp/src/main/java/io/grpc/okhttp/ExceptionHandlingFrameWriter.java
5	83.02	-9.43%	../core/src/main/java/io/grpc/internal/Http2Ping.java
11	94.2	4.26%	../netty/src/main/java/io/grpc/netty/GrpcHttp2HeadersUtils.java

Jobs

ID	Job ID	Ran	Files	Coverage
1	21691.1 (GRADLE_OPTS=-Xmx512m LDFLAGS=-L/tmp/protobuf/lib CXXFLAGS=-I/tmp/protobuf/include LD_LIBRARY_PATH=/tmp/protobuf/lib)	18 Mar 2021 11:13PM UTC	430	87.07	Travis Job 21691.1

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

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 21691

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

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 21691

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

README BADGES
x