bnhf / openvpn-admin-plus

Docker-based web interface (with golang backend) for monitoring and admin of an OpenVPN TAP/TUN server setup with PiVPN or other OpenVPN server installations. This project has been renamed from pivpn-tap-web-ui, to reflect its new broader scope.
MIT License
141 stars 23 forks source link

remove unused PIVPN_SERVER from compose env #62

Open FrancYescO opened 1 year ago

FrancYescO commented 1 year ago

better awk match for PIVPN_SERVER CN var better handling of HTTPS vars default PIVPN_SERVER to server.conf

karabelnikov commented 11 months ago

PIVPN_SERVER=${PIVPN_SERVER} and PIVPN_CONF=${PIVPN_CONF} These are required variables for openvpn-admin-plus to work. Also, the server.conf file may have a different name for others, depending on their scenarios. It is better to leave it as a variable rather than a static value for all kinds of scenarios.

FrancYescO commented 11 months ago

For conf I've just set a default value (like username and password to have a congruence in default values) and not removed the normal function when setted, for PIVPN_CONF value is already not used was just leaved in documentation, it uses some awk magic to match the value (that I've improved in this PR)