Updates the template to include a local host bind #17

Merged
merged 1 commit into from Aug 18, 2015

Conversation

Projects
None yet
3 participants
Collaborator

chuckbutler commented Aug 18, 2015

We dont wish to expose ETCD to the public interface, however the etcd
utlity appears to default to wanting to communicate over 127.0.0.1:4001.
It does expose the -C and --peers option to reconfigure itself to
communicate with a peer on the given ip:port combination, but this
breaks user expectation of things "just working".

This fix adjusts the binding which can be verified via netstat -nlp that
hte service is only listening on private address, and the adveritsing to
the API/Peers only displays the private address, while allowing the
etcdctl client to communicate using the localhost binding default.

Updates the template to include a local host bind
We dont wish to expose ETCD to the public interface, however the etcd
utlity appears to default to wanting to communicate over 127.0.0.1:4001.
It does expose the -C and --peers option to reconfigure itself to
communicate with a peer on the given ip:port combination, but this
breaks user expectation of things "just working".

This fix adjusts the binding which can be verified via netstat -nlp that
hte service is only listening on private address, and the adveritsing to
the API/Peers only displays the private address, while allowing the
etcdctl client to communicate using the localhost binding default.

Lgtm

Owner

whitmo commented Aug 18, 2015

LGTM.

whitmo added a commit that referenced this pull request Aug 18, 2015

Merge pull request #17 from chuckbutler/local-binding-fix
Updates the template to include a local host bind

@whitmo whitmo merged commit 1ef525b into whitmo:master Aug 18, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment