roster-server 2.2.9 → 2.2.10

package/index.js

@@ -247,12 +247,12 @@ class Roster {
         this.disableWildcard = options.disableWildcard !== undefined
             ? parseBooleanFlag(options.disableWildcard, false)
             : parseBooleanFlag(process.env.ROSTER_DISABLE_WILDCARD, false);
-        const combineDefault = isBunRuntime;
+        const combineDefault = false;
         this.combineWildcardCerts = options.combineWildcardCerts !== undefined
             ? parseBooleanFlag(options.combineWildcardCerts, combineDefault)
             : parseBooleanFlag(process.env.ROSTER_COMBINE_WILDCARD_CERTS, combineDefault);
         if (isBunRuntime && this.combineWildcardCerts) {
-            log.info('Bun runtime detected: combined wildcard certificates enabled by default (SNI bypass)');
+            log.info('Bun runtime detected: combined wildcard certificates enabled (SNI bypass)');
         }
 
         const port = options.port === undefined ? 443 : options.port;

package/package.json

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

package/tasks/lessons.md

@@ -1,4 +1,4 @@
 # Lessons Learned
 
 - When wildcard TLS must run under Bun, do not rely on manual DNS instructions; default to API-driven DNS-01 TXT creation/removal (Linode/Akamai) with propagation polling, then fall back to manual mode only when no provider token is configured.
-- Prefer the simplest robust DNS verification path first (dig against 1.1.1.1/8.8.8.8) and avoid adding extra resolver complexity unless there is clear evidence it improves reliability.
+- Do not keep speculative resolver/workaround attempts that are not the root cause; if a change does not resolve the issue with evidence, revert/simplify immediately so temporary experiments do not become permanent complexity.

package/test/roster-server.test.js

@@ -8,7 +8,6 @@ const http = require('http');
 const os = require('os');
 const Roster = require('../index.js');
 const {
-    isBunRuntime,
     wildcardRoot,
     hostMatchesWildcard,
     wildcardSubjectForHost,
@@ -325,14 +324,14 @@ describe('Roster', () => {
                 else process.env.ROSTER_COMBINE_WILDCARD_CERTS = previous;
             }
         });
-        it('defaults combineWildcardCerts based on isBunRuntime', () => {
+        it('defaults combineWildcardCerts to false', () => {
             const roster = new Roster({ local: false });
-            assert.strictEqual(roster.combineWildcardCerts, isBunRuntime);
-        });
-        it('explicit combineWildcardCerts=false overrides Bun default', () => {
-            const roster = new Roster({ local: false, combineWildcardCerts: false });
             assert.strictEqual(roster.combineWildcardCerts, false);
         });
+        it('explicit combineWildcardCerts=true enables combined cert mode', () => {
+            const roster = new Roster({ local: false, combineWildcardCerts: true });
+            assert.strictEqual(roster.combineWildcardCerts, true);
+        });
     });
 
     describe('register (normal domain)', () => {