haugene / vpn-configs-contrib

A collection of configs for various VPN providers
GNU General Public License v3.0
180 stars 744 forks source link

Automatic script to add to internal config #112

Closed derekcentrico closed 1 month ago

derekcentrico commented 2 years ago

Do you have the latest provider files?

Have you tested the provider files?

Can you create a PR for this config?

Provider details

Please see https://github.com/derekcentrico/vpn-configs-contrib-ovpnwork/tree/main/openvpn/ovpn and my talking-to-myself discussion at https://github.com/haugene/vpn-configs-contrib/discussions/111.

I have a script to automatically process the OVPN options based upon defined environmental variables.

I cannot test/utilize this without it being baked internally it would seem:

No bundled config script found for OVPN. Defaulting to external config Downloading configs from https://github.com/derekcentrico/vpn-configs-contrib-ovpnwork/archive/main.zip into /tmp/tmp.z3yLk1v32P Extracting configs to /tmp/tmp.Ykzp4V9zrB Found configs for OVPN in /tmp/tmp.Ykzp4V9zrB/vpn-configs-contrib-ovpnwork-main/openvpn/ovpn, will replace current content in /etc/openvpn/ovpn Cleanup: deleting /tmp/tmp.z3yLk1v32P and /tmp/tmp.Ykzp4V9zrB No VPN configuration provided. Using default. Modifying /etc/openvpn/ovpn/default.ovpn for best behaviour in this container

It may be of value for the project to verify upon download if there is a configure-openvpn.sh prior to determining internal vs. external. Currently, it won't read the OVPN script to set a default.ovpn based upon the defined variables because it isn't "internal." I can open an issue/request on the main project for this if needed.

Anything else?

No response

derekcentrico commented 2 years ago

So, I suppose that there would need to be a push of the OVPN directory contents to https://github.com/haugene/docker-transmission-openvpn/tree/master/openvpn. @haugene I am happy to maintain this if possible to obtain such a push so that ovpn is internal and the script (with the defined vars) can benefit folks more easily.

derekcentrico commented 1 month ago

Super old, long since done. Closing.