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

Fixes #103 #119

Closed
wants to merge 7 commits into from
Closed

Fixes #103 #119

wants to merge 7 commits into from

Conversation

sv0
Copy link
Contributor

@sv0 sv0 commented Aug 16, 2020

What was a problem?

The problem was reported and discussed in ticket #103

Particularly, in my comment.

How this PR fixes the problem?

  • put Nipe-specific Tor settings into /etc/torrc.d/nipe file
  • include /etc/torrc.d/nipe file into /etc/tor/torrc while running nipe.pl install
  • do not run custom Tor process, but use Tor system service instead (see Case 2 of my comment )

Check lists (check x in [ ] of list items)

  • Test passed (soon)
  • Coding style (indentation, etc)

Additional Comments

  • use "127.192.0.0/10" instead of "10.66.0.0/255.255.0.0". It's default Tor VirtualAddrNetwork
    • see man 1 tor for details
  • move startTor and stopTor commands to lib/Nipe/Utils/Device.pm
    • adjust them according to distribution in a single place in Device.pm
  • do not get Arch Linux as default distribution(and pacman as well) but show error message instead and recommend to install dependencies(tor, iptables) manualy.

@sv0 sv0 changed the title Issue #103 Fixes #103 Aug 28, 2020
@sv0 sv0 closed this Apr 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants