roster-server 2.3.4 → 2.3.8

package/README.md

@@ -309,12 +309,13 @@ await roster.start();
 const url = roster.getUrl('example.com');
 console.log(url); 
 // Local mode: http://localhost:9465
+// Local subdomain: http://api.localhost:9465
 // Production mode: https://example.com
 ```
 
 This method:
 - Returns the correct URL based on your environment (`local: true/false`)
-- In **local mode**: Returns `http://localhost:{port}` with the assigned port
+- In **local mode**: Returns `http://localhost:{port}` for apex domains and `http://{subdomain}.localhost:{port}` for subdomains
 - In **production mode**: Returns `https://{domain}` (or with custom port if configured)
 - Handles `www.` prefix automatically (returns same URL)
 - Returns `null` for domains that aren't registered
@@ -327,9 +328,12 @@ import Roster from 'roster-server';
 // Local development
 const localRoster = new Roster({ local: true });
 localRoster.register('example.com', handler);
+localRoster.register('api.example.com', handler);
 await localRoster.start();
 console.log(localRoster.getUrl('example.com')); 
 // → http://localhost:9465
+console.log(localRoster.getUrl('api.example.com'));
+// → http://api.localhost:7342
 
 // Production
 const prodRoster = new Roster({ local: false });

package/docs/decisions/local-subdomain-localhost-url.md

@@ -0,0 +1,12 @@
+---
+id: wrxv04lnq9
+type: decision
+title: 'Decision: Local subdomain URL format'
+created: '2026-03-16 17:15:01'
+---
+# Decision: Local subdomain URL format
+
+**What**: In local mode, `getUrl(domain)` now returns `http://<subdomain>.localhost:<port>` for subdomains, while apex domains keep `http://localhost:<port>`.
+**Where**: Implemented in `index.js` (`getUrl` local branch), validated in `test/roster-server.test.js`, and documented in `README.md`.
+**Why**: Local URLs should preserve subdomain semantics so multi-tenant/subdomain routes are explicit and easier to test.
+**Alternatives rejected**: Keeping all local URLs as `http://localhost:<port>` was rejected because it hides host intent for subdomain-based apps.

package/docs/generated/git-greenlock-notify-fallback-v2-3-4.md

@@ -0,0 +1,7 @@
+---
+id: oyzcq7vmk6
+type: work-log
+title: Committed and pushed greenlock notify fix
+created: '2026-03-16 15:55:18'
+---
+Created commit bfdb456 on master and pushed to origin/master. Changes: improved notify() detail-to-message normalization in index.js to handle Error objects, undefined/non-serializable inputs, and fallback to `[event] (no details)`; bumped package version to 2.3.4; added docs/generated/greenlock-notify-empty-details.md.

package/docs/generated/greenlock-notify-domain-context.md

@@ -0,0 +1,12 @@
+---
+id: blpic0aj8a
+type: bugfix
+title: 'Bugfix: Include domain in Greenlock notify logs'
+created: '2026-03-16 16:25:57'
+---
+# Bug: Greenlock notify errors lacked domain context
+
+**Symptom**: Timeout/certificate errors in `notify` logs did not clearly indicate which domain triggered the failure.
+**Root cause**: `notify` logged only the normalized message and omitted available domain fields from Greenlock/ACME details.
+**Solution**: Added domain extraction in `notify` from `subject`, `servername`, `domain`, `hostname`, `host`, `altnames`, `domains`, and `identifier.value`; prepends `[domain:<value>]` to log messages.
+**Location**: index.js Greenlock `notify` callback.

package/index.js

@@ -606,7 +606,22 @@ class Roster {
         if (this.local) {
             const pattern = resolved.siteKey.split(':')[0];
             if (this.domainPorts && this.domainPorts[pattern] !== undefined) {
-                return `http://localhost:${this.domainPorts[pattern]}`;
+                const labels = cleanDomain.split('.').filter(Boolean);
+                let localHost = 'localhost';
+
+                // For wildcard matches, preserve the full left-side subdomain chain.
+                if (pattern.startsWith('*.')) {
+                    const root = wildcardRoot(pattern);
+                    if (root && hostMatchesWildcard(cleanDomain, pattern)) {
+                        const subdomain = cleanDomain.slice(0, -(root.length + 1));
+                        if (subdomain) localHost = `${subdomain}.localhost`;
+                    }
+                } else if (labels.length > 2) {
+                    // For exact subdomains (e.g. api.example.com), map to api.localhost.
+                    localHost = `${labels.slice(0, -2).join('.')}.localhost`;
+                }
+
+                return `http://${localHost}:${this.domainPorts[pattern]}`;
             }
             return null;
         }
@@ -736,6 +751,33 @@ class Roster {
             cluster: this.cluster,
             staging: this.staging,
             notify: (event, details) => {
+                const eventDomain = (() => {
+                    if (!details || typeof details !== 'object') return null;
+
+                    const directKeys = ['subject', 'servername', 'domain', 'hostname', 'host'];
+                    for (const key of directKeys) {
+                        if (typeof details[key] === 'string' && details[key].trim()) {
+                            return details[key].trim().toLowerCase();
+                        }
+                    }
+
+                    if (Array.isArray(details.altnames) && details.altnames.length > 0) {
+                        const alt = details.altnames.find(name => typeof name === 'string' && name.trim());
+                        if (alt) return alt.trim().toLowerCase();
+                    }
+
+                    if (Array.isArray(details.domains) && details.domains.length > 0) {
+                        const domain = details.domains.find(name => typeof name === 'string' && name.trim());
+                        if (domain) return domain.trim().toLowerCase();
+                    }
+
+                    if (details.identifier && typeof details.identifier.value === 'string' && details.identifier.value.trim()) {
+                        return details.identifier.value.trim().toLowerCase();
+                    }
+
+                    return null;
+                })();
+
                 let msg;
                 if (typeof details === 'string') {
                     msg = details;
@@ -751,6 +793,9 @@ class Roster {
                     }
                 }
                 if (!msg || msg === 'undefined') msg = `[${event}] (no details)`;
+                if (eventDomain && !msg.includes(`[${eventDomain}]`)) {
+                    msg = `[domain:${eventDomain}] ${msg}`;
+                }
                 // Suppress known benign warnings from ACME when using acme-dns-01-cli
                 if (event === 'warning' && typeof msg === 'string') {
                     if (/acme-dns-01-cli.*(incorrect function signatures|deprecated use of callbacks)/i.test(msg)) return;

package/package.json

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

package/test/roster-server.test.js

@@ -218,7 +218,7 @@ describe('Roster', () => {
             roster.register('*.example.com', () => {});
             roster.domainPorts = { '*.example.com': 9999 };
             roster.local = true;
-            assert.strictEqual(roster.getUrl('api.example.com'), 'http://localhost:9999');
+            assert.strictEqual(roster.getUrl('api.example.com'), 'http://api.localhost:9999');
         });
         it('returns https URL for wildcard-matched host in production', () => {
             const roster = new Roster({ local: false });
@@ -359,6 +359,13 @@ describe('Roster', () => {
             roster.local = true;
             assert.strictEqual(roster.getUrl('exact.local'), 'http://localhost:4567');
         });
+        it('returns http://subdomain.localhost:PORT in local mode for exact subdomain', () => {
+            const roster = new Roster({ local: true });
+            roster.register('api.example.com', () => {});
+            roster.domainPorts = { 'api.example.com': 5678 };
+            roster.local = true;
+            assert.strictEqual(roster.getUrl('api.example.com'), 'http://api.localhost:5678');
+        });
         it('returns https URL in production for registered domain', () => {
             const roster = new Roster({ local: false });
             roster.register('example.com', () => {});