wfg / docker-openvpn-client

OpenVPN client with killswitch and proxy servers; built on Alpine
MIT License
353 stars 107 forks source link

fix: unbind variables #116

Open juev opened 6 months ago

juev commented 6 months ago

File entry.sh contains a number of undeclared variables. When accessing them, if they do not have a specific value, we received an error stating that a non-existent variable is being accessed.

This behavior is defined in the file header.

In order to correct this behavior, additional initialization of all variables was performed. If they are passed to the script, this value will be used. If not, the default values will be used. That is, empty values for all but KILL_SWITCH.

We also updated the alpine image to the latest stable version. And fixed the example of a docker-compose file referenced from the README.

Also, the AUTH_SECRET variable now contains the full path to the user data file. This will allow you to control the file location more precisely. And added a check, if the path to the file is set, we check that it is there before starting the openvpn process.