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

August 5 commit causes error and fails #42

Closed
foopex opened this issue Aug 9, 2017 · 2 comments
Closed

August 5 commit causes error and fails #42

foopex opened this issue Aug 9, 2017 · 2 comments

Comments

@foopex
Copy link

foopex commented Aug 9, 2017

I pulled the august 5 commit, and I get an error using the following command line(works with prior commits):
sudo openpyn us -u -f --skip-dns-patch --allow 22 80 443 3128 8112 58846 -t 10 -T 2 -d --p2p

Error:

Flushing iptables INPUT and OUTPUT chains AND Applying default Rules
...SNIP...
Top 2 Servers with best Ping are: ['us650', 'us505']
Traceback (most recent call last):
  File "/usr/local/bin/openpyn", line 11, in <module>
    load_entry_point('openpyn==1.6.0', 'console_scripts', 'openpyn')()
  File "/usr/local/lib/python3.5/dist-packages/openpyn-1.6.0-py3.5.egg/openpyn/openpyn.py", line 118, in main
    args.skip_dns_patch)
  File "/usr/local/lib/python3.5/dist-packages/openpyn-1.6.0-py3.5.egg/openpyn/openpyn.py", line 203, in run
    vpn_server_ip = get_vpn_server_ip(chosen_servers, port)
  File "/usr/local/lib/python3.5/dist-packages/openpyn-1.6.0-py3.5.egg/openpyn/openpyn.py", line 527, in get_vpn_server_ip
    file_path = "/usr/share/openpyn/files/" + server + ".nordvpn.com." + port + ".ovpn"
TypeError: Can't convert 'list' object to str implicitly
jotyGill added a commit that referenced this issue Aug 9, 2017
when the cached sudo credentials in the shell automatically expire,
needed sys.exit to cleanly exit.
@jotyGill
Copy link
Owner

jotyGill commented Aug 9, 2017

yeah, my bad. I was playing with the feature of iterating through a list of servers if connection to one fails and openvpn exits. Should be all good now.

@foopex
Copy link
Author

foopex commented Aug 9, 2017

Back to normal, thanks!

@jotyGill jotyGill closed this as completed Aug 9, 2017
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

2 participants