Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix certmagic usage. Fixes https://github.com/joohoi/acme-dns/issues/337 #351

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

maddes-b
Copy link

@maddes-b maddes-b commented Apr 11, 2024

No separate certmagic cache needed. Default config and cache are sufficient to handle the certificates for the HTTP API.
Updated to certmagic v0.20.

Issue of double cache was introduced by 27e8251 (from #334), when updating certmagic to v0.17, and reported in #337

Succesfully Tested (Ubuntu 22.04 LTS):

  • normal renewal
  • revoked cert renewal

Built on Ubuntu 22.04 LTS: GOPATH=/tmp/go-acme-dns CGO_ENABLED=1 go build (CGO_ENABLED for sqlite)

Installation on Ubuntu 22.04 LTS:

install -v -t /usr/local/bin/ acme-dns
install -v -d /etc/acme-dns/
install -v -t /etc/acme-dns/ config.cfg

adduser --system --gecos "acme-dns Service" --disabled-password --group --home /var/lib/acme-dns acme-dns
install -v -m u=rw-s,go=r-s -t /etc/systemd/system/ acme-dns.service
systemctl daemon-reload

systemctl enable acme-dns.service
systemctl start acme-dns.service

No separate certmagic cache needed. Default config and cache are sufficient
to handle the certificates for the HTTP API.
Updated to certmagic v0.20.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant