1
88%
dev: 88%

Ran 03 Aug 2017 11:51PM UTC

Files 44

Run time 2s

Badge

Embed ▾

Committed 03 Aug 2017 11:47PM UTC coverage: 88.394% (+0.4%) from 87.955%

Job # 3326.1

Build Type

push

travis-ci

Committed by

prashantv

Commit Message

Add tchannel.WithoutHeaders (#637)

This adds a function `tchannel.WithoutHeaders` (name TBD), which hides
TChannel Go-specific information from a Context.

This is needed to alleviate the following issue:

Existing TChannel Go services have been using the standard header
propagation behavior of TChannel Go (incoming headers are copied to
outgoing headers) happily and they start using YARPC+TChannel, or a
library that uses YARPC, to make outgoing calls to a different service.

Their expectation would be to have the headers copied over to that
outgoing call, especially if the underlying transport is TChannel. This
will not be supported in YARPC.

That leaves us with two options: silently ignore TChannel headers, or
reject outgoing requests made with TChannel contexts.

If we silently ignore TChannel headers, service owners who rely on this
behavior will suddenly start seeing hard-to-debug breakages. The number
of support tickets for this will be huge.

A better option is for YARPC to reject TChannel contexts on **all**
transports. YARPC will recommend that users remove the header
information from the context (with the use of `tchannel.WithoutHeaders`)
before passing it to YARPC. In the short-term, this adds a little work
for existing TChannel services but in the long run it saves them a lot
of time in terms of debugging.

Run Details

4364 of 4937 relevant lines covered (88.39%)

5977.56 hits per line

Source Files on job 3326.1

Coverage	∆	File	Lines	Relevant	Covered	Missed	Hits/Line

uber / tchannel-go / 3326 / 1
88%
dev: 88%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3326.1

uber / tchannel-go / 3326 / 1 88% dev: 88%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3326.1

uber / tchannel-go / 3326 / 1
88%
dev: 88%

README BADGES
x