@morojs/moro 1.8.4 → 1.8.5

package/dist/core/http/http-server.js

@@ -1142,7 +1142,9 @@ export class MoroHttpServer {
             httpReq.method === MoroHttpServer.METHOD_PATCH;
         const hookManager = this.hookManager;
         const hasHooks = !!(hookManager &&
-            (hookManager.hasHooks === undefined || hookManager.hasHooks('request')));
+            (hookManager.hasHooks === undefined ||
+                hookManager.hasHooks('request') ||
+                hookManager.hasHooks('response')));
         if (needsBody || hasHooks || this.globalMiddleware.length > 0) {
             // Slow path: real awaits ahead (body parse / hooks / middleware).
             // Errors are fully handled inside - the returned promise never rejects.
@@ -1190,6 +1192,14 @@ export class MoroHttpServer {
                     request: httpReq,
                     response: httpRes,
                 });
+                // Post-response hooks (session persistence, auth session refresh,
+                // CDN invalidation) run after the response is flushed, off the
+                // request's critical path.
+                if (this.hookManager.hasHooks === undefined || this.hookManager.hasHooks('response')) {
+                    httpRes.once('finish', () => {
+                        void Promise.resolve(this.hookManager.execute('response', { request: httpReq, response: httpRes })).catch((error) => this.logger?.error?.(`Response hook error: ${error?.message || error}`, 'Hooks'));
+                    });
+                }
             }
             // executeMiddleware returns undefined when the whole chain completed
             // synchronously, so a sync chain costs zero promise allocations.
@@ -1273,6 +1283,13 @@ export class MoroHttpServer {
     // Shared error handling for all request paths (sync throw, slow-path catch,
     // and promise rejections from async handlers/middleware). Never throws.
     async onRequestError(error, httpReq, httpRes, req) {
+        // Error hooks are observational (Sentry-style reporting, audit trails):
+        // fire-and-forget so a slow or throwing observer can never delay or
+        // alter the error response.
+        if (this.hookManager &&
+            (this.hookManager.hasHooks === undefined || this.hookManager.hasHooks('error'))) {
+            void Promise.resolve(this.hookManager.execute('error', { request: httpReq, response: httpRes, error })).catch((hookError) => this.logger?.error?.(`Error hook error: ${hookError?.message || hookError}`, 'Hooks'));
+        }
         try {
             // Debug: Log the actual error and where it came from
             this.logger.debug('Request error details', 'RequestHandler', {