TykTechnologies / tyk / 6321 / 2

Ran 01 Mar 2019 05:13PM UTC

Files 108

Run time 6min

Badge

Embed ▾

Committed 01 Mar 2019 05:03PM UTC coverage: 62.309% (-0.04%) from 62.349%

Job # LATEST_GO=true

Build Type

push

travis-ci

Committed by

buger

Commit Message

Fix broken logstash UDP support (#2141)

This PR disables `gas` reconnection functionality when `"logstash_transport": "udp"` because it doesn't support `udp`.
 
Also, we need to fix doc here: https://tyk.io/docs/log-data/#a-name-aggregated-logs-with-logstash-a-aggregated-logs-with-logstash. It says `logstash_transport: The Logstash transport to use, should be "tcp".` but we support `udp` too. And, it would be easier for user if we give a logstash input configuration there like:

```
input {
	tcp {	
		port => 5044
		codec => json
	}
}
```
Fixes #2136

Run Details

10509 of 16866 relevant lines covered (62.31%)

0.69 hits per line

TykTechnologies / tyk / 6321 / 2

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 6321.2 (LATEST_GO=true)

TykTechnologies / tyk / 6321 / 2

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 6321.2 (LATEST_GO=true)

README BADGES
x