@increase21/simplenodejs 1.1.0 → 1.1.1

package/dist/router.js

@@ -19,19 +19,23 @@ async function route(req, res) {
     const meta = controllers.get(controllerPath);
     if (!meta || !meta.name || !meta.Controller)
         return (0, helpers_1.throwHttpError)(404, "The requested resource does not exist");
-    const ControllerClass = meta.Controller;
-    const controller = new ControllerClass();
-    methodName = (methodName || "").replace(/\-{1}\w{1}/g, match => match.replace("-", "").toUpperCase());
-    // Block Object.prototype methods and __private convention
-    if (methodName.startsWith("__") || UNSAFE_METHODS.has(methodName)) {
-        return (0, helpers_1.throwHttpError)(404, "The requested resource does not exist");
-    }
     const ctx = {
         req, res,
         body: req.body,
         query: req.query,
         customData: req._custom_data,
     };
+    const ControllerClass = meta.Controller;
+    const controller = new ControllerClass(ctx);
+    //if request has ended, do not proceed
+    if (res.writableEnded)
+        return;
+    //sanitize method name, convert kebab-case to camelCase
+    methodName = (methodName || "").replace(/\-{1}\w{1}/g, match => match.replace("-", "").toUpperCase());
+    // Block Object.prototype methods and __private convention
+    if (methodName.startsWith("__") || UNSAFE_METHODS.has(methodName)) {
+        return (0, helpers_1.throwHttpError)(404, "The requested resource does not exist");
+    }
     // Fallback to index if method not found (treat path segment as id)
     if (typeof controller[methodName] !== "function") {
         if (typeof controller["index"] === "function" && parts.length === 3) {
@@ -47,10 +51,13 @@ async function route(req, res) {
         return (0, helpers_1.throwHttpError)(404, "Resource not found");
     }
     const descriptors = await controller[methodName](ctx, ...id);
+    // If the controller method has already sent a response, do not proceed
     if (res.writableEnded)
         return;
+    //if the handler returns no descriptors or an invalid format, end the response
     if (!descriptors || !Array.isArray(descriptors))
         return res.end();
+    // Find the descriptor matching the HTTP method
     const descriptor = descriptors.find((d) => d.method === httpMethod);
     if (!descriptor)
         return (0, helpers_1.throwHttpError)(405, "Method Not Allowed");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@increase21/simplenodejs",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "Lightweight Node.js HTTP framework with middlewares and plugins",
   "dev": "dist/index.js",
   "bugs": "https://github.com/increase21/simplenodejs/issues",
@@ -40,4 +40,4 @@
   "devDependencies": {
     "typescript": "^5.3.0"
   }
-}
+}