acme-client: use timegm() instead of mktime()
authortb <tb@openbsd.org>
Thu, 15 Dec 2022 17:36:56 +0000 (17:36 +0000)
committertb <tb@openbsd.org>
Thu, 15 Dec 2022 17:36:56 +0000 (17:36 +0000)
commit4cf348cf21299ea743a82d6708b64b5c9b12da81
tree7c53eb7e66ca1615cf561869b9d80eed0c27351a
parentbeb45aa65eb6a84603e029e1a375e59f63112a68
acme-client: use timegm() instead of mktime()

Times in certificates are all expressed in Zulu time, so calling the time
zone dependent mktime() on such a time and comparing it to time(NULL) is
wrong.

This means that the check of at least 30 days validity and deciding on
whether to renew or not might have been off by by half a day depending on
where you are. That should not matter since you (or cron) are supposed to
run acme-client way more often than once a month.

ok claudio millert
usr.sbin/acme-client/revokeproc.c