acme: Fix bugs from upstream's change to socat
The upstream acme.sh package changed to using socat instead of netcat; update the dependencies to reflect this, and pass --listen-v6 when running in standalone mode (since socat only listens on IPv4 by default). Also add a missing cleanup call when certificate issuance fails. Signed-off-by: Toke Høiland-Jørgensen <toke@toke.dk>
This commit is contained in:
parent
2a7e216188
commit
56b6d49ce4
2 changed files with 4 additions and 3 deletions
|
@ -9,7 +9,7 @@ include $(TOPDIR)/rules.mk
|
||||||
|
|
||||||
PKG_NAME:=acme
|
PKG_NAME:=acme
|
||||||
PKG_VERSION:=2.7.9
|
PKG_VERSION:=2.7.9
|
||||||
PKG_RELEASE:=1
|
PKG_RELEASE:=2
|
||||||
PKG_LICENSE:=GPLv3
|
PKG_LICENSE:=GPLv3
|
||||||
|
|
||||||
PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.gz
|
PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.gz
|
||||||
|
@ -25,7 +25,7 @@ include $(INCLUDE_DIR)/package.mk
|
||||||
define Package/acme
|
define Package/acme
|
||||||
SECTION:=net
|
SECTION:=net
|
||||||
CATEGORY:=Network
|
CATEGORY:=Network
|
||||||
DEPENDS:=+curl +ca-bundle +openssl-util +netcat
|
DEPENDS:=+curl +ca-bundle +openssl-util +socat
|
||||||
TITLE:=ACME (Letsencrypt) client
|
TITLE:=ACME (Letsencrypt) client
|
||||||
endef
|
endef
|
||||||
|
|
||||||
|
|
|
@ -187,7 +187,7 @@ issue_cert()
|
||||||
acme_args="$acme_args --dns $dns"
|
acme_args="$acme_args --dns $dns"
|
||||||
elif [ -z "$webroot" ]; then
|
elif [ -z "$webroot" ]; then
|
||||||
log "Using standalone mode"
|
log "Using standalone mode"
|
||||||
acme_args="$acme_args --standalone"
|
acme_args="$acme_args --standalone --listen-v6"
|
||||||
else
|
else
|
||||||
if [ ! -d "$webroot" ]; then
|
if [ ! -d "$webroot" ]; then
|
||||||
err "$main_domain: Webroot dir '$webroot' does not exist!"
|
err "$main_domain: Webroot dir '$webroot' does not exist!"
|
||||||
|
@ -211,6 +211,7 @@ issue_cert()
|
||||||
err "Restoring staging certificate"
|
err "Restoring staging certificate"
|
||||||
mv "$STATE_DIR/${main_domain}.staging" "$STATE_DIR/${main_domain}"
|
mv "$STATE_DIR/${main_domain}.staging" "$STATE_DIR/${main_domain}"
|
||||||
fi
|
fi
|
||||||
|
post_checks
|
||||||
return 1
|
return 1
|
||||||
fi
|
fi
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue