3162

Ran 02 Oct 2017 06:25PM UTC

Jobs 1

Files 84

Run time 4s

Badge

Embed ▾

pending completion

Build # 3162

Build Type

push

travis-ci

Committed by

buger

Commit Message

proxy: fix and test multi-target reverse proxy

I broke it when replacing the false Init method with a regular
constructor, since I forgot to replace the nil return with the actual
object.

Add a test that covers both the default proxy and an overriding one. It
was crashing as expected. Fix the constructor, making the test pass.

While at it, clean up the code quite a bit:

* no need to parse TargetURL again
* getProxyForRequest can be much simpler
* unindent some of the code

Fixes #1177.

Coverage Stats

6107 of 13300 relevant lines covered (45.92%)

0.5 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
2	3162.2 (LATEST_GO=true)	02 Oct 2017 06:25PM UTC	0	45.92	Travis Job 3162.2

Source Files on build 3162

Detailed source file information is not available for this build.

TykTechnologies / tyk / 3162

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 3162

TykTechnologies / tyk / 3162

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 3162

README BADGES
x