mastercontroller 1.3.16 → 1.3.18

package/MasterRouter.js

@@ -89,7 +89,9 @@ const ROUTER_CONFIG = {
                 const sanitizedValue = sanitizeRouteParam(paramName, paramValue);
 
                 requestParams[paramName] = sanitizedValue;
-                routePathList[i] = sanitizedValue;
+                // FIX: Use requestItem (original value) not sanitizedValue for route matching
+                // The route pattern should match against the actual request path
+                routePathList[i] = requestItem;
             }
         }
     }
@@ -906,7 +908,32 @@ class MasterRouter {
                 throw new TypeError('Request object must have a valid response property');
             }
 
+            // Normalize pathName from various request object formats
             if (!rr.pathName || typeof rr.pathName !== 'string') {
+                // Try to extract pathName from standard HTTP request properties
+                if (rr.request.url) {
+                    // Handle full URL or path with query string (Node.js http, Express)
+                    try {
+                        // Try parsing as full URL first
+                        const url = new URL(rr.request.url, `http://${rr.request.headers?.host || 'localhost'}`);
+                        rr.pathName = url.pathname;
+                    } catch (e) {
+                        // Fallback: treat as relative path, strip query string
+                        rr.pathName = rr.request.url.split('?')[0];
+                    }
+                } else if (rr.request.path) {
+                    // Express-style request.path
+                    rr.pathName = rr.request.path;
+                } else if (rr.request.pathname) {
+                    // Alternative property name
+                    rr.pathName = rr.request.pathname;
+                } else {
+                    throw new TypeError('Request object must have a valid path (url, path, pathname, or pathName property)');
+                }
+            }
+
+            // Validate that we now have a valid pathName
+            if (typeof rr.pathName !== 'string') {
                 throw new TypeError('Request object must have a valid pathName');
             }
 
@@ -915,7 +942,9 @@ class MasterRouter {
             }
 
             const $that = this;
-            const requestObject = Object.create(rr);
+            // FIX: Use direct reference instead of Object.create() to preserve request/response objects
+            // Object.create() puts properties on prototype, causing undefined access issues
+            const requestObject = rr;
 
             // CRITICAL FIX: Load scoped services into request-specific context
             // Pass requestObject so scoped services are stored per-request, not globally

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mastercontroller",
-  "version": "1.3.16",
+  "version": "1.3.18",
   "description": "Fortune 500 ready Node.js MVC framework with enterprise security, monitoring, and horizontal scaling",
   "main": "MasterControl.js",
   "license": "MIT",