Improved check for base64 options

pieterlange / kube-openvpn

:closed_lock_with_key: Kubernetes native OpenVPN

MIT License

383 stars 57 forks source link

Improved check for base64 options #47

Closed andrew-nguyen closed 7 years ago

andrew-nguyen commented 7 years ago

Previous version checked if the script was being run in Linux. This tests the actual command to see if -w0 is accepted since some may use GNU's base64 on non-Linux platforms (e.g., MacOS)

pieterlange commented 7 years ago

Seems to make sense to me! Merging.

Note that i still eventually want to fix this deployment process to be more portable by either adopting helm and/or moving to the kubernetes CustomResource / operator design pattern, but it will take some time.

Edit: and obviously: thanks! :-)