14401
89%

Ran 18 Jun 2019 04:52PM UTC

Jobs 1

Files 261

Run time 16s

Badge

Embed ▾

pending completion

Build # 14401

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

netty,alts: fire initial protocol negotiation event in WBAEH

This change is needed after trying to use the new style protocol negotiators internally.  The problem is that some handlers fire the event in handlerAdded, which is too early.  The followup PNE is fired after handlerAdded, which breaks the composibility of the negotiators.

To fix this, this change modifies the negotiation flow.  Specifically:

* Negotiators should NEVER fire a negotiation from handlerAdded, instead they should wait until userEventTriggered
* Negotiators now do state checking on the PNE.  If it is set twice, it fails.  If it has not been received when doing the next stage of negotiation, it fails.
* WBAEH now fires the initial, default event.  This is the only handler that can fire it from handlerAdded

The tests updated are ones not using WBAEH (which they probably should).  This change ensures attributes aren't lost when doing negotiation.

Run Details

15040 of 17235 relevant lines covered (87.26%)

0.87 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	14401.1	18 Jun 2019 04:52PM UTC	0	87.26	Travis Job 14401.1

Source Files on build 14401

Detailed source file information is not available for this build.

grpc / grpc-java / 14401
89%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 14401

grpc / grpc-java / 14401 89%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 14401

grpc / grpc-java / 14401
89%

README BADGES
x