fix: don't attempt renewing acme if domain is not specified
Massimo Melina committed
Dec 23, 2023 at 19:33 UTC
74cb7e054346d2d7a0e19a751d64271e09620671
1 file changed
+3
-2
src/acme.ts
+3
-2
@@ -99,7 +99,8 @@ events.once('https ready', () => repeat(HOUR, renewCert))
99
100
// checks if the cert is near expiration date, and if so renews it
101
const renewCert = debounceAsync(async () => {
102
- if (!acmeRenew.get()) return
102
+ const domain = acmeDomain.get()
103
+ if (!acmeRenew.get() || !domain) return
104
const cert = getCertObject()
105
if (!cert) return
106
const now = new Date()
@@ -107,7 +108,7 @@ const renewCert = debounceAsync(async () => {
108
// not expiring in a month
109
if (now > new Date(cert.validFrom) && now < validTo && validTo.getTime() - now.getTime() >= 30 * DAY)
110
return console.log("certificate still good")
110
- await makeCert(acmeDomain.get(), acmeEmail.get())
111
+ await makeCert(domain, acmeEmail.get())
112
.catch(e => console.log("error renewing certificate: ", String(e)))
113
}, 0, { retain: DAY, retainFailure: HOUR })
114