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

Use os.devnull #43

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Use os.devnull #43

wants to merge 1 commit into from

Conversation

addshore
Copy link

@addshore addshore commented May 24, 2016

This is a step toward making this run on Windows.
This now means at least the help output can be
seen on Windows.

This is a setp toward making this run on Windows.
This now means at least the help output can be
seen on Windows.
AevaOnline pushed a commit to AevaOnline/sshuttle that referenced this pull request Sep 10, 2016
In some cases (see apenwarr#43) it seems that some network configurations may
end up setting a skip on lo. As sshuttle adds rules that rely on
filtering/translating packets on lo, this causes problem. This fix
overrides the skip and makes the rules be applied again.
Should fix at least some of the problems reported on apenwarr#43.
Copy link

@b-long b-long left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems like a useful change, and easy enough to merge. 👍

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

Successfully merging this pull request may close these issues.

2 participants