Added TigerVPN as a new Provider #28
Conversation
added Tiger vpn as the new OPENVPN provider
| @@ -4,9 +4,12 @@ if [ "$OPENVPN_PROVIDER" = "BTGUARD" ] | |||
| then | |||
| echo "VPN PROVIDER: BTGUARD" | |||
| vpn_provider="btguard" | |||
| else | |||
| else if [ "$OPENVPN_PROVIDER" = "PIA" ] | |||
There was a problem hiding this comment.
For backwards compatibility PIA should probably be the "default" VPN provider and any new ones should be called out explicitly.
There was a problem hiding this comment.
Might not be a bad idea to just make $OPENVPN_PROVIDER a required input when running the container.
There was a problem hiding this comment.
I agree @dougnukem . Thinking something like:
if provider = btguard
else if provider = pia
else if ...
else echo "No valid provider given"; exit 1
fi
ish?
This will also be backward compatible as OPENVPN_PROVIDER is set to PIA in Dockerfile. Could of course remove the default and make provider a required input as well. Defaulting to PIA makes less sense when the number of providers grows...
Added TigerVPN as a new Provider
|
hey sorry i did a spell error have opened a pull request with the correct filename |
|
I didn't see it either. Merged now 👍 |
added Tiger vpn as the new OPENVPN provider