@tahminator/sapling 1.2.0 → 1.3.1

package/dist/src/annotation/controller.js

@@ -34,7 +34,7 @@ export function Controller({ prefix = "", deps = [] } = {
         _usedPrefixes.add(prefix);
         const router = Router();
         const routes = _getRoutes(target);
-        const usedPaths = new Set();
+        const usedRoutes = new Set();
         _InjectableDeps.set(targetClass, deps);
         const controllerInstance = _resolve(targetClass);
         for (const { method, path, fnName } of routes) {
@@ -42,13 +42,14 @@ export function Controller({ prefix = "", deps = [] } = {
             if (typeof fn !== "function")
                 continue;
             const fp = prefix + path;
+            const routeKey = method + " " + fp;
             // Only check for duplicates on non-middleware routes
-            // Middleware (USE) can have duplicate paths
-            if (method !== "USE" && usedPaths.has(fp)) {
-                throw new Error(`Duplicate route path "${fp}" detected in controller "${target.name}"`);
+            // Middleware (USE) can have duplicate paths, and different HTTP methods can share paths
+            if (method !== "USE" && usedRoutes.has(routeKey)) {
+                throw new Error(`Duplicate route [${method}] "${fp}" detected in controller "${target.name}"`);
             }
             if (method !== "USE") {
-                usedPaths.add(fp);
+                usedRoutes.add(routeKey);
             }
             const methodName = methodResolve[method];
             router[methodName](fp, (request, response, next) => __awaiter(this, void 0, void 0, function* () {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tahminator/sapling",
-  "version": "1.2.0",
+  "version": "1.3.1",
   "description": "A library to help you write cleaner Express.js code",
   "scripts": {
     "test": "echo \"Error: no test specified\" && exit 1",