roster-server 2.3.8 → 2.3.10

package/demo/www/api.example.com/index.js

@@ -1,6 +1,6 @@
 module.exports = (httpsServer) => {
     return (req, res) => {
         res.writeHead(200, { 'Content-Type': 'text/plain; charset=utf-8' });
-        res.end('"subdomain.example.com: Crazy from thinking, wanting to be reasonable, and the heart has reasons that reason itself will never understand."');
+        res.end('"api.example.com: Crazy from thinking, wanting to be reasonable, and the heart has reasons that reason itself will never understand."');
     };
 };

package/demo/www/express.example.com/index.js

@@ -4,7 +4,7 @@ module.exports = (httpsServer) => {
     const app = express();
     app.get('/', (req, res) => {
         res.setHeader('Content-Type', 'text/plain; charset=utf-8');
-        res.send('"subdomain.example.com: Loco de pensar, queriendo entrar en razón, y el corazón tiene razones que la propia razón nunca entenderá."');
+        res.send('"express.example.com: Loco de pensar, queriendo entrar en razón, y el corazón tiene razones que la propia razón nunca entenderá."');
     });
 
     return app;

package/docs/generated/local-mode-log-hostname-format.md

@@ -0,0 +1,12 @@
+---
+id: fralm92cih
+type: bugfix
+title: 'Bugfix: local mode startup logs show subdomain.localhost'
+created: '2026-03-16 17:19:26'
+---
+# Bug: local mode startup logs ignored subdomain localhost format
+
+**Symptom**: Startup logs still printed `http://localhost:<port>` for subdomains (e.g. `api.example.com`) even after local URL behavior changed.
+**Root cause**: `startLocalMode()` log message used a hardcoded `localhost` string.
+**Solution**: Added `localHostForDomain(domain)` helper and reused it in both `getUrl()` and local-mode startup logs so they stay consistent.
+**Location**: `index.js` (`localHostForDomain`, `getUrl`, `startLocalMode` listen log).

package/index.js

@@ -109,6 +109,19 @@ function parseBooleanFlag(value, fallback = false) {
     return fallback;
 }
 
+function normalizeDomainForLocalHost(domain) {
+    return (domain || '').trim().toLowerCase().replace(/^www\./, '');
+}
+
+function localHostForDomain(normalizedDomain) {
+    const normalized = normalizedDomain;
+    if (!normalized) return 'localhost';
+    if (normalized.startsWith('*.')) return '*.localhost';
+    const labels = normalized.split('.').filter(Boolean);
+    if (labels.length > 2) return `${labels.slice(0, -2).join('.')}.localhost`;
+    return 'localhost';
+}
+
 // Virtual Server that completely isolates applications
 class VirtualServer extends EventEmitter {
     constructor(domain) {
@@ -597,7 +610,7 @@ class Roster {
      * @returns {string|null} The URL if domain is registered (exact or wildcard), null otherwise
      */
     getUrl(domain) {
-        const cleanDomain = domain.startsWith('www.') ? domain.slice(4) : domain;
+        const cleanDomain = normalizeDomainForLocalHost(domain);
 
         const exactMatch = this.sites[cleanDomain] || this.sites[`www.${cleanDomain}`];
         const resolved = exactMatch ? { handler: exactMatch, siteKey: cleanDomain } : this.getHandlerAndKeyForHost(cleanDomain);
@@ -606,22 +619,7 @@ class Roster {
         if (this.local) {
             const pattern = resolved.siteKey.split(':')[0];
             if (this.domainPorts && this.domainPorts[pattern] !== undefined) {
-                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 `http://${localHostForDomain(cleanDomain)}:${this.domainPorts[pattern]}`;
             }
             return null;
         }
@@ -719,7 +717,8 @@ class Roster {
             });
 
             httpServer.listen(port, 'localhost', () => {
-                log.info(`🌐 ${domain} → http://localhost:${port}`);
+                const cleanDomain = normalizeDomainForLocalHost(domain);
+                log.info(`🌐 ${domain} → http://${localHostForDomain(cleanDomain)}:${port}`);
             });
 
             httpServer.on('error', (error) => {
@@ -794,7 +793,7 @@ class Roster {
                 }
                 if (!msg || msg === 'undefined') msg = `[${event}] (no details)`;
                 if (eventDomain && !msg.includes(`[${eventDomain}]`)) {
-                    msg = `[domain:${eventDomain}] ${msg}`;
+                    msg = `[${eventDomain}] ${msg}`;
                 }
                 // Suppress known benign warnings from ACME when using acme-dns-01-cli
                 if (event === 'warning' && typeof msg === 'string') {

package/package.json

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

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

@@ -1,12 +0,0 @@
----
-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.