@lucaapp/service-utils 5.29.0 → 5.29.1

package/dist/lib/api/openapi.js

@@ -225,6 +225,27 @@ const hoistCombinatorRequiredFields = (spec) => {
     visit(schemas);
 };
 exports.hoistCombinatorRequiredFields = hoistCombinatorRequiredFields;
+/**
+ * Turn each middleware's declared schemes into OpenAPI security requirements.
+ *
+ * OpenAPI expresses the two relations differently: separate objects in the
+ * `security` array are alternatives (OR), while several schemes inside one
+ * object are all required (AND). A middleware's own `security: [a, b]` means
+ * "a or b" -- it accepts either credential -- but two middlewares on the same
+ * route both run, so their requirements are combined with AND.
+ *
+ * That makes the result the cartesian product across middlewares:
+ *
+ *   [[a, b], [c]]  ->  [{a, c}, {b, c}]
+ *
+ * A single middleware therefore still renders as plain alternatives, which is
+ * what it did before, and stacked middlewares no longer render as if presenting
+ * any one of their credentials were enough.
+ */
+const buildSecurityRequirements = (securityByMiddleware) => securityByMiddleware.reduce((requirements, alternatives) => requirements.flatMap(requirement => 
+// Duplicates collapse on their own: the same scheme required twice is
+// still one entry in the object.
+alternatives.map(scheme => ({ ...requirement, [scheme]: [] }))), [{}]);
 const registerEndpoint = (registry, method, prefixPath, rootPrefixPath, path, summary, 
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 options, tags, debug) => {
@@ -233,7 +254,11 @@ options, tags, debug) => {
     let querySchema = options.schemas?.query || undefined;
     let headersSchema = options.schemas?.headers || undefined;
     let responseSchemas = options.responses;
-    const securitySchemes = new Set();
+    // One entry per security-declaring middleware, each holding that middleware's
+    // alternatives. Kept separate because the two levels mean different things:
+    // within a middleware the schemes are alternatives, across middlewares they
+    // are all enforced. See buildSecurityRequirements.
+    const securityByMiddleware = [];
     if (options.errors) {
         for (const errorType of Object.keys(options.errors)) {
             const statusCode = options.errors[errorType];
@@ -294,8 +319,9 @@ options, tags, debug) => {
                 });
             }
         }
-        for (const scheme of middleware.options.security || []) {
-            securitySchemes.add(scheme);
+        const middlewareSecurity = middleware.options.security || [];
+        if (middlewareSecurity.length > 0) {
+            securityByMiddleware.push([...middlewareSecurity]);
         }
     }
     // Add ZodError response
@@ -355,8 +381,8 @@ options, tags, debug) => {
         responses: responseMap,
         operationId: options.operationId ||
             deriveOperationId(registry, method, prefixPath.slice(rootPrefixPath.length) + path),
-        ...(securitySchemes.size > 0 && {
-            security: [...securitySchemes].map(scheme => ({ [scheme]: [] })),
+        ...(securityByMiddleware.length > 0 && {
+            security: buildSecurityRequirements(securityByMiddleware),
         }),
     });
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lucaapp/service-utils",
-  "version": "5.29.0",
+  "version": "5.29.1",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "files": [