Skip to content
This repository has been archived by the owner on Mar 9, 2022. It is now read-only.

listener: Changed succesful HTTP response code from 200 to 204 #78

Merged
merged 1 commit into from
May 15, 2018
Merged

listener: Changed succesful HTTP response code from 200 to 204 #78

merged 1 commit into from
May 15, 2018

Conversation

oplehto
Copy link
Contributor

@oplehto oplehto commented May 15, 2018

Telegraf HTTP InfluxDB output throws an error if Spout HTTP listener returns a 200 instead of 204 (StatusNoContent). The latter is the standard "OK" response from InfluxDB.

 E! InfluxDB Output Error: Response Error: Status Code [200], expected [204], [<nil>]

@oplehto oplehto requested a review from mjs May 15, 2018 07:38
@oplehto oplehto changed the title Changed HTTP listener to return 204 instead of 200 listener: Changed succesful HTTP response code from 200 to 204 May 15, 2018
Copy link
Contributor

@mjs mjs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mjs mjs merged commit 5c8fc8a into jumptrading:master May 15, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants