Skip to content

Commit

Permalink
easy-rsa: Drop all mention of the vars file
Browse files Browse the repository at this point in the history
* This exists to import/export a number of easy-rsa default values but
  creates headaches for old volumes due to changes where easy-rsa
  insists on loading the var file if the environment variable is set.
* Going forward people should pass the variables via:
    `docker run -e EASYRSA_var ...`
* Closes kylemanna#608
  • Loading branch information
Kyle Manna committed Dec 4, 2020
1 parent c4b9436 commit 6ad9310
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 46 deletions.
3 changes: 1 addition & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,7 @@ RUN echo "http://dl-cdn.alpinelinux.org/alpine/edge/testing/" >> /etc/apk/reposi
ENV OPENVPN=/etc/openvpn
ENV EASYRSA=/usr/share/easy-rsa \
EASYRSA_CRL_DAYS=3650 \
EASYRSA_PKI=$OPENVPN/pki \
EASYRSA_VARS_FILE=$OPENVPN/vars
EASYRSA_PKI=$OPENVPN/pki

VOLUME ["/etc/openvpn"]

Expand Down
1 change: 0 additions & 1 deletion Dockerfile.aarch64
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ RUN echo "http://dl-4.alpinelinux.org/alpine/edge/community/" >> /etc/apk/reposi
ENV OPENVPN /etc/openvpn
ENV EASYRSA /usr/share/easy-rsa
ENV EASYRSA_PKI $OPENVPN/pki
ENV EASYRSA_VARS_FILE $OPENVPN/vars

# Prevents refused client connection because of an expired CRL
ENV EASYRSA_CRL_DAYS 3650
Expand Down
39 changes: 0 additions & 39 deletions bin/easyrsa_vars

This file was deleted.

4 changes: 0 additions & 4 deletions bin/ovpn_initpki
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,6 @@ source "$OPENVPN/ovpn_env.sh"
# Specify "nopass" as arg[2] to make the CA insecure (not recommended!)
nopass=$1

# EasyRSA 3.0.7 introduced checks for $EASYRSA_VARS_FILE existence
# in the init-pki script
touch $EASYRSA_VARS_FILE

# Provides a sufficient warning before erasing pre-existing files
easyrsa init-pki

Expand Down

0 comments on commit 6ad9310

Please sign in to comment.