-
Notifications
You must be signed in to change notification settings - Fork 12
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
NordVPN version #4
Comments
Cool. I'm glad that the bulk of the script worked for Nord. Thanks for sharing hartzan! Now time to think of a good way to merge these additions :) |
Could you paste the output of |
Here you are: nordvpn status nordvpn status |
I'm going to work on a small python program to:
The script could then be called inside |
@sudo-julia Sounds good! Yeah, that'd be clean. Seems like that would clean up about half the differences in code between Mullvad and Nord and set the script up for future services. We have these basic utility differences, shown below, which I was thinking may be best kept in a setup file the user could run to swap out relevant pieces with
And then there is the regional information, which if I understand, your suggestion should let us manage much more efficiently.
Thanks! |
Hey! Being able to swap variables with a script sounds like a really good idea. I think it's great to have options that are able to be configured without having to open the main script, which could be daunting for newer users even though the way you laid it out is super accessible. If you check my fork the python script is up and running, I just got lost at where to integrate it in |
Hi,
here https://gist.github.com/hartzan/b85644cb30799c98a449814c234f6953 you can find my version for NordVPN. Geoip stuff is not needed, NordVPN status message contains IP,country and city. Unfortunately there is no 2-letter country code so it might happen that all letters are not visible? I guess it's not perfect but works for me :-)
The text was updated successfully, but these errors were encountered: