Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Allow binding to a specific interface #45

Merged
merged 2 commits into from Feb 11, 2012

Conversation

Projects
None yet
3 participants
Contributor

bluesmoon commented Oct 28, 2011

statsd currently binds to all (IPv4) interfaces. This may not always be what we want
(it's not what I want). Sometimes we just want to bind to a private interface so that
only boxes on our local network can send us messages.

This patch lets us do that.

Note that we default to undefined. Both dgram.js and net.js treat undefined as the
default value, ie, bind to everything.

@bluesmoon bluesmoon Allow binding to a specific interface
statsd currently binds to all (IPv4) interfaces.  This may not always be what we want
(it's not what I want).  Sometimes we just want to bind to a private interface so that
only boxes on our local network can send us messages.

This patch lets us do that.

Note that we default to undefined.  Both dgram.js and net.js treat undefined as the
default value, ie, bind to everything.
c2ab6c9

yuvadm commented Nov 2, 2011

+1 on this, would like to see this issue merged.

This is a dupe of #20, but a cleaner patch.

@kastner kastner added a commit that referenced this pull request Feb 11, 2012

@kastner kastner Merge pull request #45 from lognormal/selective-iface
Allow binding to a specific interface
990d4df

@kastner kastner merged commit 990d4df into etsy:master Feb 11, 2012

Contributor

kastner commented Feb 11, 2012

Awesome, thanks

@hbouvier hbouvier pushed a commit to hbouvier/statsd that referenced this pull request May 25, 2013

@kastner kastner Merge pull request #45 from lognormal/selective-iface
Allow binding to a specific interface
ca90060
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment