Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nodemanager should listen on localhost #90

Open
choksi81 opened this issue Jun 2, 2014 · 1 comment
Open

Nodemanager should listen on localhost #90

choksi81 opened this issue Jun 2, 2014 · 1 comment
Assignees

Comments

@choksi81
Copy link
Contributor

choksi81 commented Jun 2, 2014

Currently, the node manager does not listen on the loopback interface (aka localhost, 127.0.0.1, or 127/8). This is irrelevant for typically donator installations, but makes accessing local vessels on a Custom Install unnecessarily difficult to access, requiring Internet connectivity and a public IP (or NAT Affixes).

The nodemanager listening on localhost would also ease our problems with running Repy locally on Android, where the usual

python repy.py restrictionsfile myprogram.repy

just doesn't work out of the box: You could use a machine connected over ADB, set up port forwarding (adb forward tcp:1224 tcp:12345), and then connect to 127.0.0.1:12345 using seash on the debug machine to reach the node manager on the Android device. (Alas, this will only become useful when we have a Custom Installer Builder for Android, #1193).

I suggest making the node manager listen on localhost per default, and possibly adding an install-time switch to disable this function.


By the way, the node manager can be told to at install time to listen ''only'' on localhost,

$ ./install.sh --nm-iface lo

but then it won't be contactable from the outside at all.

@choksi81 choksi81 self-assigned this Jun 2, 2014
@choksi81
Copy link
Contributor Author

choksi81 commented Jun 3, 2014

I think you're really saying the default Affix should contain a branch that allows receiving connection on local host.
I'm fine with this. I don't know we need a code change in the NM or elsewhere for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant