@trojs/openapi-server 3.6.0 → 3.7.0

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@trojs/openapi-server",
   "description": "OpenAPI Server",
-  "version": "3.6.0",
+  "version": "3.7.0",
   "author": {
     "name": "Pieter Wigboldus",
     "url": "https://trojs.org/"
@@ -40,7 +40,7 @@
     "@trojs/lint": "^0.4.0",
     "@types/node": "^24.0.0",
     "@types/express-serve-static-core": "^5.0.0",
-    "eslint": "^9.15.0",
+    "eslint": "^10.0.0",
     "globals": "^17.0.0",
     "jscpd": "^4.0.5",
     "prettier": "^3.3.3",

package/src/handlers/not-found.js

@@ -1,5 +1,7 @@
 export const notFound = (_context, request, response) => {
-  response.status(404)
+  if (!response.headersSent) {
+    response.status(404)
+  }
   return {
     status: 404,
     timestamp: new Date(),

package/src/handlers/request-validation.js

@@ -1,5 +1,7 @@
 export const requestValidation = (context, request, response) => {
-  response.status(400)
+  if (!response.headersSent) {
+    response.status(400)
+  }
   return {
     errors: context.validation.errors,
     status: 400,

package/src/handlers/response-validation.js

@@ -1,13 +1,22 @@
-export default (logger) => (context, request, response) => {
+export default (logger, validateResponse) => (context, request, response) => {
+  // Prevent sending response if headers are already sent (e.g., after redirect)
+  if (response.headersSent) {
+    return undefined
+  }
+
   const responseDoesntNeedValidation = response.statusCode >= 400
   if (responseDoesntNeedValidation) {
     return response.json(context.response)
   }
 
-  const valid = context.api.validateResponse(
-    context.response,
-    context.operation
-  )
+  const valid = validateResponse
+    ? (
+        context.api.validateResponse(
+          context.response,
+          context.operation
+        )
+      )
+    : null
   if (valid?.errors) {
     if (logger) {
       logger.error({
@@ -18,12 +27,15 @@ export default (logger) => (context, request, response) => {
         response: context.response
       })
     }
-    return response.status(502).json({
-      errors: valid.errors,
-      status: 502,
-      timestamp: new Date(),
-      message: 'Bad response'
-    })
+    if (!response.headersSent) {
+      return response.status(502).json({
+        errors: valid.errors,
+        status: 502,
+        timestamp: new Date(),
+        message: 'Bad response'
+      })
+    }
+    return undefined
   }
 
   if (!context.response) {

package/src/handlers/unauthorized.js

@@ -1,5 +1,7 @@
 export const unauthorized = async (context, request, response) => {
-  response.status(401)
+  if (!response.headersSent) {
+    response.status(401)
+  }
   return {
     status: 401,
     timestamp: new Date(),

package/src/router.js

@@ -66,9 +66,7 @@ export const setupRouter = ({
     notFound
   }
 
-  if (validateResponse) {
-    handlers.postResponseHandler = makeResponseValidation(logger)
-  }
+  handlers.postResponseHandler = makeResponseValidation(logger, validateResponse)
 
   api.register(handlers)
 

package/types/handlers/not-found.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"not-found.d.ts","sourceRoot":"","sources":["../../src/handlers/not-found.js"],"names":[],"mappings":"AAAO;;;;EAON"}
+{"version":3,"file":"not-found.d.ts","sourceRoot":"","sources":["../../src/handlers/not-found.js"],"names":[],"mappings":"AAAO;;;;EASN"}

package/types/handlers/request-validation.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"request-validation.d.ts","sourceRoot":"","sources":["../../src/handlers/request-validation.js"],"names":[],"mappings":"AAAO;;;;;EAQN"}
+{"version":3,"file":"request-validation.d.ts","sourceRoot":"","sources":["../../src/handlers/request-validation.js"],"names":[],"mappings":"AAAO;;;;;EAUN"}

package/types/handlers/response-validation.d.ts

@@ -1,3 +1,3 @@
-declare function _default(logger: any): (context: any, request: any, response: any) => any;
+declare function _default(logger: any, validateResponse: any): (context: any, request: any, response: any) => any;
 export default _default;
 //# sourceMappingURL=response-validation.d.ts.map

package/types/handlers/response-validation.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"response-validation.d.ts","sourceRoot":"","sources":["../../src/handlers/response-validation.js"],"names":[],"mappings":"AAAe,yCAAa,YAAO,EAAE,YAAO,EAAE,aAAQ,SAsCrD"}
+{"version":3,"file":"response-validation.d.ts","sourceRoot":"","sources":["../../src/handlers/response-validation.js"],"names":[],"mappings":"AAAe,gEAA+B,YAAO,EAAE,YAAO,EAAE,aAAQ,SAkDvE"}

package/types/handlers/unauthorized.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"unauthorized.d.ts","sourceRoot":"","sources":["../../src/handlers/unauthorized.js"],"names":[],"mappings":"AAAO;;;;GAON"}
+{"version":3,"file":"unauthorized.d.ts","sourceRoot":"","sources":["../../src/handlers/unauthorized.js"],"names":[],"mappings":"AAAO;;;;GASN"}

package/types/router.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"router.d.ts","sourceRoot":"","sources":["../src/router.js"],"names":[],"mappings":"AAmCO,sNAfJ;IAAuB,oBAAoB,EAAnC,MAAM;IACS,WAAW,EAA1B,MAAM;IACU,OAAO,GAAvB,MAAM,YAAC;IACU,mBAAmB,GAApC,OAAO,YAAC;IACS,YAAY,GAA7B,OAAO,YAAC;IACQ,MAAM,GAAtB,MAAM,YAAC;IACS,IAAI,GAApB,MAAM,YAAC;IACoB,gBAAgB,GAA3C,eAAe,EAAE,YAAC;IACD,mBAAmB,GAApC,OAAO,YAAC;IACS,gBAAgB,GAAjC,OAAO,YAAC;IACS,UAAU,GAA3B,OAAO,YAAC;IACe,YAAY,GAAnC,aAAa,YAAC;IACG,IAAI,GAArB,OAAO,YAAC;CAChB,GAAU;IAAE,GAAG,EAAE,cAAc,CAAC,GAAG,CAAC,CAAC;IAAC,oBAAoB,EAAE,MAAM,CAAA;CAAE,CA0EtE;qBAjGY,OAAO,UAAU,EAAE,MAAM;8BACzB,OAAO,UAAU,EAAE,eAAe;sBAClC,OAAO,UAAU,EAAE,OAAO;sBAC1B,OAAO,KAAK,EAAE,OAAO;4BACrB,OAAO,iBAAiB,EAAE,aAAa;+BAdrB,iBAAiB"}
+{"version":3,"file":"router.d.ts","sourceRoot":"","sources":["../src/router.js"],"names":[],"mappings":"AAmCO,sNAfJ;IAAuB,oBAAoB,EAAnC,MAAM;IACS,WAAW,EAA1B,MAAM;IACU,OAAO,GAAvB,MAAM,YAAC;IACU,mBAAmB,GAApC,OAAO,YAAC;IACS,YAAY,GAA7B,OAAO,YAAC;IACQ,MAAM,GAAtB,MAAM,YAAC;IACS,IAAI,GAApB,MAAM,YAAC;IACoB,gBAAgB,GAA3C,eAAe,EAAE,YAAC;IACD,mBAAmB,GAApC,OAAO,YAAC;IACS,gBAAgB,GAAjC,OAAO,YAAC;IACS,UAAU,GAA3B,OAAO,YAAC;IACe,YAAY,GAAnC,aAAa,YAAC;IACG,IAAI,GAArB,OAAO,YAAC;CAChB,GAAU;IAAE,GAAG,EAAE,cAAc,CAAC,GAAG,CAAC,CAAC;IAAC,oBAAoB,EAAE,MAAM,CAAA;CAAE,CAwEtE;qBA/FY,OAAO,UAAU,EAAE,MAAM;8BACzB,OAAO,UAAU,EAAE,eAAe;sBAClC,OAAO,UAAU,EAAE,OAAO;sBAC1B,OAAO,KAAK,EAAE,OAAO;4BACrB,OAAO,iBAAiB,EAAE,aAAa;+BAdrB,iBAAiB"}