2679

Ran 26 Jul 2017 02:23PM UTC

Jobs 1

Files 84

Run time 4s

Badge

Embed ▾

pending completion

Build # 2679

Build Type

push

travis-ci

Committed by

buger

Commit Message

mw: avoid double goroutine in rate limiting

If UseAsyncSessionWrite is true, DefaultSessionManager.UpdateSession
does the actual store write in a goroutine. Other than that, the
function has very little logic and a json marshal.

As such, it's redundant for this middleware to also use a goroutine. It
would only lead to extra CPU use and delay in writing the session to the
store, given how two goroutines need to be scheduled instead of just
one.

Coverage Stats

5939 of 13576 relevant lines covered (43.75%)

0.47 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	2679.1 (LATEST_GO=true)	26 Jul 2017 02:23PM UTC	0	43.75	Travis Job 2679.1

Source Files on build 2679

Detailed source file information is not available for this build.

TykTechnologies / tyk / 2679

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 2679

TykTechnologies / tyk / 2679

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 2679

README BADGES
x