Bind to localhost only when start listening #57

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants

As we are relying on local URL scheme for testing anyway, there is no need to expose our test server to the world (at least when starting it ourselves).

Member

gjohnson commented Sep 29, 2013

Since attaching to an already active server is supported, I would recommend going that route, if binding to INADDR_ANY is an issue. Node-core uses this default too for net/http listen(), so I think best to follow their pattern here.

@gjohnson gjohnson closed this Sep 29, 2013

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