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

Entry_DNS component toggle for Node #437

Open
kauri-hero opened this issue Apr 21, 2024 · 3 comments · May be fixed by #537
Open

Entry_DNS component toggle for Node #437

kauri-hero opened this issue Apr 21, 2024 · 3 comments · May be fixed by #537
Labels

Comments

@kauri-hero
Copy link
Contributor

kauri-hero commented Apr 21, 2024

Currently when the MASQ starts, it always runs with entry_dns on and port 53 is listened to by Node.

Design a way to allow a startup parameter to be toggled that will turn off entry_dns and not require Node to listen on port 53, so other software can listen on the port unobstructed.

*Consider removing the entry_dns component, if system wide subversion is never to be a "thing"

Present a log to user to remind them that they will need to proxy network traffic/software to MASQ for privacy routing

ALSO check the daemon to ensure that entry_dns is not starting when daemon is running - there is a chance that this was turned on during some

SUGGESTIONS from Dan:

  • Leave the entry_dns server in MASQNode for now, just in case it becomes useful in a system too small to have extensive proxying powers.
  • In ServerInitializerReal, make the dns_socket_server field an Option. Start it out as None.
  • Modify ServerInitializerReal::go() so that it needs a particular value from the multi_config before it will construct self.dns_socket_server and call .initialize_as_privileged() and .initialize_as_unprivileged() on it. That way, unless the user specifies something like --entry-dns on, there will be no contention over Port 53.
  • Put help in shared_schema.rs that explains to the user the consequences of both --entry-dns on and --entry-dns off (which should be the default).

Command for inspecting processes with MASQNode
macOS
ps -ef | grep MASQNode (includes this command)

window
netstat -ano

List processes listening on port 53:
macOS
lsof -Pn -i :53

@kauri-hero kauri-hero moved this to Ready for Development in MASQ Node v2 May 19, 2024
@dnwiebe
Copy link
Collaborator

dnwiebe commented May 20, 2024

Suggestions:

  • Leave the entry_dns server in MASQNode for now, just in case it becomes useful in a system too small to have extensive proxying powers.
  • In ServerInitializerReal, make the dns_socket_server field an Option. Start it out as None.
  • Modify ServerInitializerReal::go() so that it needs a particular value from the multi_config before it will construct self.dns_socket_server and call .initialize_as_privileged() and .initialize_as_unprivileged() on it. That way, unless the user specifies something like --entry-dns on, there will be no contention over Port 53.
  • Put help in shared_schema.rs that explains to the user the consequences of both --entry-dns on and --entry-dns off (which should be the default).

@kauri-hero kauri-hero added the duplicate This issue or pull request already exists label May 28, 2024
@kauri-hero kauri-hero closed this as not planned Won't fix, can't repro, duplicate, stale May 28, 2024
@github-project-automation github-project-automation bot moved this from 🔖 Ready to ✅ Done in MASQ Node v2 May 28, 2024
@kauri-hero kauri-hero moved this from ✅ Done to 🔖 Ready in MASQ Node v2 May 28, 2024
@kauri-hero kauri-hero added story and removed duplicate This issue or pull request already exists labels May 28, 2024
@kauri-hero
Copy link
Contributor Author

Other card duplicate to this is:
MASQ-Project/MASQ-Node-issues#153

@kauri-hero kauri-hero reopened this May 28, 2024
@kauri-hero
Copy link
Contributor Author

this is somewhat related to old card #446

@kauri-hero kauri-hero moved this from 🔖 Ready to 🔖 Awaiting Development (Prioritized) in MASQ Node v2 Aug 4, 2024
@github-project-automation github-project-automation bot moved this to Done in MASQNode Aug 29, 2024
This was referenced Oct 7, 2024
@utkarshg6 utkarshg6 linked a pull request Oct 9, 2024 that will close this issue
@utkarshg6 utkarshg6 moved this from 🔖 Awaiting Development (Prioritized) to 👀 Review + QA in Progress in MASQ Node v2 Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 👀 Review + QA in Progress
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants