From 47cc0e3ae6b295a01f03516c3902b5ff081ec27c Mon Sep 17 00:00:00 2001 From: Robin Schneider Date: Sat, 14 Mar 2015 13:22:28 +0100 Subject: [PATCH] Fixed based on the review by @kylemanna. Thanks. --- README.md | 4 ++-- bin/ovpn_getclient | 34 +++++++++++++++++----------------- docs/advanced.md | 5 ++--- docs/clients.md | 8 ++++---- 4 files changed, 25 insertions(+), 26 deletions(-) diff --git a/README.md b/README.md index 785c5c4..f717227 100644 --- a/README.md +++ b/README.md @@ -37,7 +37,7 @@ Upstream links: * Retrieve the client configuration with embedded certificates - docker run --volumes-from $OVPN_DATA --rm kylemanna/openvpn ovpn_getclient CLIENTNAME combined > CLIENTNAME.ovpn + docker run --volumes-from $OVPN_DATA --rm kylemanna/openvpn ovpn_getclient CLIENTNAME > CLIENTNAME.ovpn * Create an environment variable with the name DEBUG and value of 1 to enable debug output (using "docker -e"). @@ -105,7 +105,7 @@ packets, etc). simplicity. It's highly recommended to secure the CA key with some passphrase to protect against a filesystem compromise. A more secure system would put the EasyRSA PKI CA on an offline system (can use the same Docker - image and the script ovpn_copy_server_files to accomplish this). + image and the script [`ovpn_copy_server_files`](/docs/clients.md) to accomplish this). * It would be impossible for an adversary to sign bad or forged certificates without first cracking the key's passphase should the adversary have root access to the filesystem. diff --git a/bin/ovpn_getclient b/bin/ovpn_getclient index 06a041e..d076c9d 100755 --- a/bin/ovpn_getclient +++ b/bin/ovpn_getclient @@ -25,7 +25,7 @@ cn="$1" parm="$2" if [ ! -f "$EASYRSA_PKI/private/${cn}.key" ]; then - >&2 "Unable to find \"${cn}\", please try again or generate the key first" 1>&2 + echo "Unable to find \"${cn}\", please try again or generate the key first" >&2 exit 1 fi @@ -39,8 +39,8 @@ remote-cert-tls server remote $OVPN_CN $OVPN_PORT $OVPN_PROTO " -if [ "$mode" == "combined" ]; then - echo " + if [ "$mode" == "combined" ]; then + echo " $(cat $EASYRSA_PKI/private/${cn}.key) @@ -58,23 +58,23 @@ $(cat $EASYRSA_PKI/ta.key) key-direction 1 " -else - echo " + elif [ "$mode" == "separated" ]; then + echo " key ${cn}.key ca ca.crt cert ${cn}.crt dh dh.pem tls-auth ta.key 1 " -fi + fi -if [ "$OVPN_DEFROUTE" != "0" ];then - echo "redirect-gateway def1" -fi + if [ "$OVPN_DEFROUTE" != "0" ];then + echo "redirect-gateway def1" + fi -if [ -n "$OVPN_MTU" ]; then - echo "tun-mtu $OVPN_MTU" -fi + if [ -n "$OVPN_MTU" ]; then + echo "tun-mtu $OVPN_MTU" + fi } dir="$OPENVPN/clients/$cn" @@ -88,16 +88,16 @@ case "$parm" in cp "$EASYRSA_PKI/dh.pem" "$dir/dh.pem" cp "$EASYRSA_PKI/ta.key" "$dir/ta.key" ;; - "combined") + "" | "combined") get_client_config "combined" ;; "combined-save") get_client_config "combined" > "$dir/${cn}-combined.ovpn" ;; *) - >&2 echo "This script can produce the client configuration in to formats." - >&2 echo " 1. combined: All needed configuration and cryptographic material is in one file (Use \"combined-save\" to write the configuration file in the same path as the separated parameter does)." - >&2 echo " 2. separated: Separated files." - >&2 echo "Please specific one of those options as second parameter." + echo "This script can produce the client configuration in to formats:" >&2 + echo " 1. combined (default): All needed configuration and cryptographic material is in one file (Use \"combined-save\" to write the configuration file in the same path as the separated parameter does)." >&2 + echo " 2. separated: Separated files." >&2 + echo "Please specific one of those options as second parameter." >&2 ;; esac diff --git a/docs/advanced.md b/docs/advanced.md index 39bda25..59cb9be 100644 --- a/docs/advanced.md +++ b/docs/advanced.md @@ -1,6 +1,6 @@ # Advanced Configurations -The ovpn_genconfig script is intended for simple configurations that apply to the majority of the users. If your use case isn't general, it likely won't be supported. This document aims to explain how to work around that. +The [`ovpn_genconfig`](/bin/ovpn_genconfig) script is intended for simple configurations that apply to the majority of the users. If your use case isn't general, it likely won't be supported. This document aims to explain how to work around that. ## Create host volume mounts rather than data volumes @@ -13,9 +13,8 @@ The ovpn_genconfig script is intended for simple configurations that apply to th docker run --rm -v $PWD:/etc/openvpn -it kylemanna/openvpn ovpn_initpki vim openvpn.conf docker run --rm -v $PWD:/etc/openvpn -it kylemanna/openvpn easyrsa build-client-full CLIENTNAME nopass - docker run --rm -v $PWD:/etc/openvpn kylemanna/openvpn ovpn_getclient CLIENTNAME combined > CLIENTNAME.ovpn + docker run --rm -v $PWD:/etc/openvpn kylemanna/openvpn ovpn_getclient CLIENTNAME > CLIENTNAME.ovpn * Start the server with: docker run -v $PWD:/etc/openvpn -d -p 1194:1194/udp --privileged kylemanna/openvpn - diff --git a/docs/clients.md b/docs/clients.md index 81f029f..5ace6c9 100644 --- a/docs/clients.md +++ b/docs/clients.md @@ -2,16 +2,16 @@ ## Client configuration mode -The `ovpn_getclient` can produce two different versions of the configuration. +The [`ovpn_getclient`](/bin/ovpn_getclient) can produce two different versions of the configuration. -1. combined: All needed configuration and cryptographic material is in one file (Use "combined-save" to write the configuration file in the same path as the separated parameter does). +1. combined (default): All needed configuration and cryptographic material is in one file (Use "combined-save" to write the configuration file in the same path as the separated parameter does). 2. separated: Separated files. Note that some client software might be picky about which configuration format it accepts. ## Batch mode -If you have more than a few clients, you will want to generate and update your client configuration in batch. For this task the script `ovpn_getclient_all` was written, which writes out the configuration for each client to a separate directory called `clients/$cn`. +If you have more than a few clients, you will want to generate and update your client configuration in batch. For this task the script [`ovpn_getclient_all`](/bin/ovpn_getclient_all) was written, which writes out the configuration for each client to a separate directory called `clients/$cn`. Execute the following to generate the configuration for all clients: @@ -21,7 +21,7 @@ After doing so, you will find the following files in each of the `$cn` directori ca.crt dh.pem - $cn-combined.ovpn # Combined configuration file format, you your client recognices this file then only this file is needed. + $cn-combined.ovpn # Combined configuration file format. If your client recognices this file then only this file is needed. $cn.ovpn # Separated configuration. This configuration file requires the other files ca.crt dh.pem $cn.crt $cn.key ta.key $cn.crt $cn.key