roster-server 2.3.12 → 2.3.16

package/index.js

@@ -258,17 +258,14 @@ class Roster {
         this.maxLocalPort = options.maxLocalPort || 9999;
         this.tlsMinVersion = options.tlsMinVersion ?? 'TLSv1.2';
         this.tlsMaxVersion = options.tlsMaxVersion ?? 'TLSv1.3';
-        this.disableWildcard = options.disableWildcard !== undefined
-            ? parseBooleanFlag(options.disableWildcard, false)
-            : parseBooleanFlag(process.env.ROSTER_DISABLE_WILDCARD, false);
-        const combineDefault = false;
-        this.combineWildcardCerts = options.combineWildcardCerts !== undefined
-            ? parseBooleanFlag(options.combineWildcardCerts, combineDefault)
-            : parseBooleanFlag(process.env.ROSTER_COMBINE_WILDCARD_CERTS, combineDefault);
+        this.disableWildcard = parseBooleanFlag(options.disableWildcard, false);
+        this.combineWildcardCerts = parseBooleanFlag(options.combineWildcardCerts, false);
         if (isBunRuntime && this.combineWildcardCerts) {
             log.info('Bun runtime detected: combined wildcard certificates enabled (SNI bypass)');
         }
 
+        this.skipLocalCheck = parseBooleanFlag(options.skipLocalCheck, true);
+
         const port = options.port === undefined ? 443 : options.port;
         if (port === 80 && !this.local) {
             throw new Error('⚠️  Port 80 is reserved for ACME challenge. Please use a different port.');
@@ -749,6 +746,8 @@ class Roster {
             maintainerEmail: this.email,
             cluster: this.cluster,
             staging: this.staging,
+            skipDryRun: this.skipLocalCheck,
+            skipChallengeTest: this.skipLocalCheck,
             notify: (event, details) => {
                 const eventDomain = (() => {
                     if (!details || typeof details !== 'object') return null;

package/package.json

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

package/test/roster-server.test.js

@@ -302,27 +302,13 @@ describe('Roster', () => {
             const roster = new Roster({ local: true, disableWildcard: true });
             assert.strictEqual(roster.disableWildcard, true);
         });
-        it('reads disableWildcard from env var', () => {
-            const previous = process.env.ROSTER_DISABLE_WILDCARD;
-            process.env.ROSTER_DISABLE_WILDCARD = '1';
-            try {
-                const roster = new Roster({ local: true });
-                assert.strictEqual(roster.disableWildcard, true);
-            } finally {
-                if (previous === undefined) delete process.env.ROSTER_DISABLE_WILDCARD;
-                else process.env.ROSTER_DISABLE_WILDCARD = previous;
-            }
+        it('enables disableWildcard from constructor option (truthy string)', () => {
+            const roster = new Roster({ local: true, disableWildcard: '1' });
+            assert.strictEqual(roster.disableWildcard, true);
         });
-        it('enables combined wildcard certs from env var', () => {
-            const previous = process.env.ROSTER_COMBINE_WILDCARD_CERTS;
-            process.env.ROSTER_COMBINE_WILDCARD_CERTS = '1';
-            try {
-                const roster = new Roster({ local: false });
-                assert.strictEqual(roster.combineWildcardCerts, true);
-            } finally {
-                if (previous === undefined) delete process.env.ROSTER_COMBINE_WILDCARD_CERTS;
-                else process.env.ROSTER_COMBINE_WILDCARD_CERTS = previous;
-            }
+        it('enables combined wildcard certs from constructor option', () => {
+            const roster = new Roster({ local: false, combineWildcardCerts: true });
+            assert.strictEqual(roster.combineWildcardCerts, true);
         });
         it('defaults combineWildcardCerts to false', () => {
             const roster = new Roster({ local: false });

package/vendor/greenlock/greenlock.js

@@ -373,7 +373,9 @@ G.create = function(gconf) {
             maintainerEmail: gconf.maintainerEmail,
             packageAgent: packageAgent,
             notify: greenlock._notify,
-            debug: greenlock._defaults.debug || args.debug
+            debug: greenlock._defaults.debug || args.debug,
+            skipDryRun: gconf.skipDryRun || false,
+            skipChallengeTest: gconf.skipChallengeTest || false
         });
 
         var dir = caches[dirUrl];
@@ -382,13 +384,34 @@ G.create = function(gconf) {
             return dir.promise;
         }
 
-        await acme.init(dirUrl).catch(function(err) {
-            log.error(
-                "ACME init failed (directory may be down or directoryUrl wrong):",
-                err.message
-            );
-            throw err;
-        });
+        var maxRetries = 3;
+        var lastErr;
+        for (var attempt = 1; attempt <= maxRetries; attempt++) {
+            try {
+                await acme.init(dirUrl);
+                lastErr = null;
+                break;
+            } catch (err) {
+                lastErr = err;
+                log.error(
+                    "ACME init attempt " +
+                        attempt +
+                        "/" +
+                        maxRetries +
+                        " failed (directory may be down or directoryUrl wrong):",
+                    err.message
+                );
+                if (attempt < maxRetries) {
+                    await new Promise(function(resolve) {
+                        setTimeout(resolve, 1000 * attempt);
+                    });
+                }
+            }
+        }
+        if (lastErr) {
+            delete caches[dirUrl];
+            throw lastErr;
+        }
 
         caches[dirUrl] = {
             promise: Promise.resolve(acme),

package/docs/decisions/network-default-bind-hostname.md

@@ -1,7 +0,0 @@
----
-id: ac8ba0ra21
-type: decision
-title: Roster default IPv6 bind
-created: '2026-03-16 21:57:18'
----
-Changed Roster default bind hostname from 0.0.0.0 to :: in index.js constructor (`this.hostname = options.hostname ?? '::'`). This makes default listen IPv6-first. Existing explicit hostname option still overrides. Verified with full test suite: 74 passing.