Skip to content
This repository has been archived by the owner on Sep 19, 2024. It is now read-only.

Add support for TCP connections (as introduced in etsy/statsd@v0.8.0) #67

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

remie
Copy link

@remie remie commented Apr 19, 2017

This PR adds support for TCP connections, which was included in the etsy/statsd library in version 0.8.0 (current). It adds a boolean 'tcp' option. If set to true, it will create a net.Socket connection with keepAlive enabled to the specified host/port. Stats are written to the TCP socket for processing by Statsd.

The unit tests have been updated to also include a TCP server test, with TCP server implementation taken from etsy/statsd.

remie added 4 commits April 19, 2017 15:02
…from the Travis CI build and adding Node 4, 5, 6 and 7 :)
Adding documentation on `tcp` option and updating reference to supported NodeJS versions
@bdeitte
Copy link

bdeitte commented Oct 3, 2017

This project hasn't been updated in years. If you'd like to submit to https://github.com/brightcove/hot-shots I'll look it over there

@honzajavorek
Copy link

@remie @bdeitte Has this been ported to hot shots in the end? I can't see anything about TCP in the hot shots' docs or code.

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.

None yet

3 participants