roster-server 2.1.27 → 2.1.30

package/index.js

@@ -258,7 +258,7 @@ class Roster {
                 provided.module = defaultDnsChallengeModule;
             }
             if (provided.propagationDelay === undefined) {
-                provided.propagationDelay = 60000;
+                provided.propagationDelay = 120000;
             }
             if (provided.autoContinue === undefined) {
                 provided.autoContinue = false;
@@ -270,9 +270,9 @@ class Roster {
         } else {
             this.dnsChallenge = {
                 module: defaultDnsChallengeModule,
-                propagationDelay: 60000,
+                propagationDelay: 120000,
                 autoContinue: false,
-                dryRunDelay: 60000
+                dryRunDelay: 120000
             };
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "roster-server",
-    "version": "2.1.27",
+    "version": "2.1.30",
     "description": "👾 RosterServer - A domain host router to host multiple HTTPS.",
     "main": "index.js",
     "scripts": {

package/vendor/acme-dns-01-cli-wrapper.js

@@ -81,6 +81,23 @@ module.exports.create = function create(config = {}) {
         : Number.isFinite(Number(process.env.ROSTER_DNS_POLL_TIMEOUT_MS))
             ? Number(process.env.ROSTER_DNS_POLL_TIMEOUT_MS)
             : null;
+    const dryRunPollTimeoutMs = Number.isFinite(config.dryRunPollTimeoutMs)
+        ? config.dryRunPollTimeoutMs
+        : Number.isFinite(Number(process.env.ROSTER_DNS_DRYRUN_POLL_TIMEOUT_MS))
+            ? Number(process.env.ROSTER_DNS_DRYRUN_POLL_TIMEOUT_MS)
+            : null;
+    const parseResolvers = (value) => String(value || '')
+        .split(',')
+        .map((s) => s.trim())
+        .filter(Boolean);
+    const configuredResolvers = Array.isArray(config.dnsResolvers)
+        ? config.dnsResolvers.map((s) => String(s).trim()).filter(Boolean)
+        : parseResolvers(process.env.ROSTER_DNS_RESOLVERS);
+    const resolverClients = configuredResolvers.map((server) => {
+        const resolver = new dns.Resolver();
+        resolver.setServers([server]);
+        return { server, resolver };
+    });
 
     function sleep(ms) {
         return new Promise((resolve) => setTimeout(resolve, ms));
@@ -95,10 +112,29 @@ module.exports.create = function create(config = {}) {
         return candidate ? String(candidate).trim() : '';
     }
 
+    async function resolveTxtRecords(dnsHost) {
+        const records = [];
+        if (resolverClients.length === 0) {
+            return await dns.resolveTxt(dnsHost);
+        }
+
+        for (const { resolver } of resolverClients) {
+            try {
+                const result = await resolver.resolveTxt(dnsHost);
+                if (Array.isArray(result)) {
+                    records.push(...result);
+                }
+            } catch (_) {
+                // Try next configured resolver
+            }
+        }
+        return records;
+    }
+
     async function hasDnsTxtToken(dnsHost, expectedToken) {
         if (!dnsHost || !expectedToken) return false;
         try {
-            const records = await dns.resolveTxt(dnsHost);
+            const records = await resolveTxtRecords(dnsHost);
             for (const recordParts of records || []) {
                 const joined = (recordParts || []).map(normalizeTxtChunk).join('').trim();
                 if (joined === expectedToken) return true;
@@ -111,10 +147,9 @@ module.exports.create = function create(config = {}) {
 
     async function waitForDnsTxtPropagation(dnsHost, expectedToken, timeoutMs) {
         const started = Date.now();
-        const maxWait = Math.max(0, Number.isFinite(timeoutMs) ? timeoutMs : 0);
-        if (maxWait === 0) return hasDnsTxtToken(dnsHost, expectedToken);
-
-        while ((Date.now() - started) <= maxWait) {
+        const hasFiniteTimeout = Number.isFinite(timeoutMs) && timeoutMs >= 0;
+        const maxWait = hasFiniteTimeout ? timeoutMs : null;
+        while (maxWait === null || (Date.now() - started) <= maxWait) {
             if (await hasDnsTxtToken(dnsHost, expectedToken)) return true;
             await sleep(Math.max(1000, dnsPollIntervalMs));
         }
@@ -142,7 +177,9 @@ module.exports.create = function create(config = {}) {
         const effectiveDelay = isDryRunChallenge
             ? Math.max(0, dryRunDelay)
             : propagationDelay;
-        const effectiveTimeoutMs = dnsPollTimeoutMs === null ? effectiveDelay : dnsPollTimeoutMs;
+        const effectiveTimeoutMs = isDryRunChallenge
+            ? dryRunPollTimeoutMs
+            : (dnsPollTimeoutMs === null ? effectiveDelay : dnsPollTimeoutMs);
         const expectedToken = resolveExpectedToken(opts, ch);
 
         log.info('DNS-01 ' + altname);
@@ -152,7 +189,7 @@ module.exports.create = function create(config = {}) {
                 'DNS verification enabled. Continuing automatically when TXT appears at ' +
                 dnsHost +
                 ' (timeout ' +
-                effectiveTimeoutMs +
+                (effectiveTimeoutMs === null ? 'infinite' : effectiveTimeoutMs) +
                 'ms, poll ' +
                 dnsPollIntervalMs +
                 'ms).'
@@ -163,7 +200,7 @@ module.exports.create = function create(config = {}) {
                 return null;
             }
             log.warn(
-                `DNS TXT not detected for ${dnsHost} within ${effectiveTimeoutMs}ms; ` +
+                `DNS TXT not detected for ${dnsHost} within ${effectiveTimeoutMs}; ` +
                 'continuing anyway (ACME preflight may still fail).'
             );
             return null;