@webpieces/http-server 0.3.382 → 0.3.384

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@webpieces/http-server",
-  "version": "0.3.382",
+  "version": "0.3.384",
   "description": "WebPieces server with filter chain and dependency injection",
   "type": "commonjs",
   "main": "./src/index.js",
@@ -22,10 +22,10 @@
     "access": "public"
   },
   "dependencies": {
-    "@webpieces/core-context": "0.3.382",
-    "@webpieces/core-util": "0.3.382",
-    "@webpieces/gcp-identity": "0.3.382",
-    "@webpieces/http-routing": "0.3.382",
+    "@webpieces/core-context": "0.3.384",
+    "@webpieces/core-util": "0.3.384",
+    "@webpieces/gcp-identity": "0.3.384",
+    "@webpieces/http-routing": "0.3.384",
     "cors": "2.8.5",
     "express": "5.1.0",
     "inversify": "7.10.4"

package/src/filters/LogApiFilter.d.ts

@@ -1,7 +1,5 @@
 import { MethodMeta } from '@webpieces/http-routing';
 import { Filter, WpResponse, Service } from '@webpieces/http-routing';
 export declare class LogApiFilter extends Filter<MethodMeta, WpResponse<unknown>> {
-    private logApiCall;
-    constructor();
     filter(meta: MethodMeta, nextFilter: Service<MethodMeta, WpResponse<unknown>>): Promise<WpResponse<unknown>>;
 }

package/src/filters/LogApiFilter.js

@@ -12,10 +12,10 @@ const core_util_2 = require("@webpieces/core-util");
  * Priority: 1800 (after ContextFilter at 2000, before custom filters)
  *
  * Logging patterns (via LogApiCall):
- * - [API-SVR-req] Class.method /url request={...} headers={...}
- * - [API-SVR-resp-SUCCESS] Class.method response={...}
- * - [API-SVR-resp-FAIL] Class.method error=... (server errors: 500, 502, 504)
- * - [API-SVR-resp-OTHER] Class.method errorType=... (user errors: 400, 401, 403, 404, 266)
+ * - [API-server-req] Class.method /url request={...}
+ * - [API-server-resp-SUCCESS] Class.method response={...}
+ * - [API-server-resp-FAIL] Class.method error=... (server errors: 500, 502, 504)
+ * - [API-server-resp-OTHER] Class.method errorType=... (user errors: 400, 401, 403, 404, 266)
  *
  * Headers are read from RequestContext (NOT from meta.requestHeaders which is undefined
  * after ContextFilter runs at priority 2000).
@@ -25,27 +25,22 @@ const core_util_2 = require("@webpieces/core-util");
  */
 const log = core_util_1.LogManager.getLogger('LogApiFilter');
 let LogApiFilter = class LogApiFilter extends http_routing_2.Filter {
-    logApiCall;
-    constructor() {
-        super();
-        // Context fields are stamped onto each record by the logging BACKEND (bunyan/winston read
-        // RequestContext.buildLogFields()); this filter no longer collects them.
-        this.logApiCall = new core_util_2.LogApiCall();
-    }
     async filter(meta, nextFilter) {
         // Wrap nextFilter.invoke in a method that returns the response
         const method = async () => {
             const wpResponse = await nextFilter.invoke(meta);
             return wpResponse.response;
         };
-        const response = await this.logApiCall.execute("SVR", meta.routeMeta, meta.requestDto, method);
+        // LogApiCall is a singleton (use it directly, no `new`). It logs the text lines AND stamps the
+        // structured `api={side:'server',...}` tag into RequestContext, so every log line during the
+        // request carries jsonPayload.api. Correlation fields (requestId, ...) are added by the backend.
+        const response = await core_util_2.LogApiCall.execute('server', meta.routeMeta, meta.requestDto, method);
         return new http_routing_2.WpResponse(response);
     }
 };
 exports.LogApiFilter = LogApiFilter;
 exports.LogApiFilter = LogApiFilter = tslib_1.__decorate([
     (0, http_routing_1.provideFrameworkSingleton)(),
-    (0, inversify_1.injectable)(),
-    tslib_1.__metadata("design:paramtypes", [])
+    (0, inversify_1.injectable)()
 ], LogApiFilter);
 //# sourceMappingURL=LogApiFilter.js.map

package/src/filters/LogApiFilter.js.map

@@ -1 +1 @@
-{"version":3,"file":"LogApiFilter.js","sourceRoot":"","sources":["../../../../../../packages/http/http-server/src/filters/LogApiFilter.ts"],"names":[],"mappings":";;;;AAAA,yCAAuC;AACvC,0DAA8E;AAC9E,0DAAsE;AACtE,oDAAkD;AAClD,oDAE8B;AAE9B;;;;;;;;;;;;;;;GAeG;AACH,MAAM,GAAG,GAAG,sBAAU,CAAC,SAAS,CAAC,cAAc,CAAC,CAAC;AAI1C,IAAM,YAAY,GAAlB,MAAM,YAAa,SAAQ,qBAAuC;IAC7D,UAAU,CAAa;IAE/B;QACI,KAAK,EAAE,CAAC;QACR,0FAA0F;QAC1F,yEAAyE;QACzE,IAAI,CAAC,UAAU,GAAG,IAAI,sBAAU,EAAE,CAAC;IACvC,CAAC;IAED,KAAK,CAAC,MAAM,CACR,IAAgB,EAChB,UAAoD;QAEpD,+DAA+D;QAC/D,MAAM,MAAM,GAAG,KAAK,IAAsB,EAAE;YACxC,MAAM,UAAU,GAAG,MAAM,UAAU,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;YACjD,OAAO,UAAU,CAAC,QAAQ,CAAC;QAC/B,CAAC,CAAC;QAEF,MAAM,QAAQ,GAAG,MAAM,IAAI,CAAC,UAAU,CAAC,OAAO,CAAC,KAAK,EAAE,IAAI,CAAC,SAAS,EAAE,IAAI,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC;QAC/F,OAAO,IAAI,yBAAU,CAAC,QAAQ,CAAC,CAAC;IACpC,CAAC;CACJ,CAAA;AAvBY,oCAAY;uBAAZ,YAAY;IAFxB,IAAA,wCAAyB,GAAE;IAC3B,IAAA,sBAAU,GAAE;;GACA,YAAY,CAuBxB","sourcesContent":["import { injectable } from 'inversify';\nimport {provideFrameworkSingleton, MethodMeta} from '@webpieces/http-routing';\nimport { Filter, WpResponse, Service } from '@webpieces/http-routing';\nimport { LogManager } from '@webpieces/core-util';\nimport {\n    LogApiCall,\n} from '@webpieces/core-util';\n\n/**\n * LogApiFilter - Structured API logging for all requests/responses.\n * Priority: 1800 (after ContextFilter at 2000, before custom filters)\n *\n * Logging patterns (via LogApiCall):\n * - [API-SVR-req] Class.method /url request={...} headers={...}\n * - [API-SVR-resp-SUCCESS] Class.method response={...}\n * - [API-SVR-resp-FAIL] Class.method error=... (server errors: 500, 502, 504)\n * - [API-SVR-resp-OTHER] Class.method errorType=... (user errors: 400, 401, 403, 404, 266)\n *\n * Headers are read from RequestContext (NOT from meta.requestHeaders which is undefined\n * after ContextFilter runs at priority 2000).\n *\n * User errors (HttpBadRequestError, etc.) are logged as OTHER, not FAIL,\n * because they are expected behavior from the server's perspective.\n */\nconst log = LogManager.getLogger('LogApiFilter');\n\n@provideFrameworkSingleton()\n@injectable()\nexport class LogApiFilter extends Filter<MethodMeta, WpResponse<unknown>> {\n    private logApiCall: LogApiCall;\n\n    constructor() {\n        super();\n        // Context fields are stamped onto each record by the logging BACKEND (bunyan/winston read\n        // RequestContext.buildLogFields()); this filter no longer collects them.\n        this.logApiCall = new LogApiCall();\n    }\n\n    async filter(\n        meta: MethodMeta,\n        nextFilter: Service<MethodMeta, WpResponse<unknown>>,\n    ): Promise<WpResponse<unknown>> {\n        // Wrap nextFilter.invoke in a method that returns the response\n        const method = async (): Promise<unknown> => {\n            const wpResponse = await nextFilter.invoke(meta);\n            return wpResponse.response;\n        };\n\n        const response = await this.logApiCall.execute(\"SVR\", meta.routeMeta, meta.requestDto, method);\n        return new WpResponse(response);\n    }\n}\n"]}
+{"version":3,"file":"LogApiFilter.js","sourceRoot":"","sources":["../../../../../../packages/http/http-server/src/filters/LogApiFilter.ts"],"names":[],"mappings":";;;;AAAA,yCAAuC;AACvC,0DAA8E;AAC9E,0DAAsE;AACtE,oDAAkD;AAClD,oDAAkD;AAElD;;;;;;;;;;;;;;;GAeG;AACH,MAAM,GAAG,GAAG,sBAAU,CAAC,SAAS,CAAC,cAAc,CAAC,CAAC;AAI1C,IAAM,YAAY,GAAlB,MAAM,YAAa,SAAQ,qBAAuC;IAErE,KAAK,CAAC,MAAM,CACR,IAAgB,EAChB,UAAoD;QAEpD,+DAA+D;QAC/D,MAAM,MAAM,GAAG,KAAK,IAAsB,EAAE;YACxC,MAAM,UAAU,GAAG,MAAM,UAAU,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;YACjD,OAAO,UAAU,CAAC,QAAQ,CAAC;QAC/B,CAAC,CAAC;QAEF,+FAA+F;QAC/F,6FAA6F;QAC7F,iGAAiG;QACjG,MAAM,QAAQ,GAAG,MAAM,sBAAU,CAAC,OAAO,CAAC,QAAQ,EAAE,IAAI,CAAC,SAAS,EAAE,IAAI,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC;QAC7F,OAAO,IAAI,yBAAU,CAAC,QAAQ,CAAC,CAAC;IACpC,CAAC;CACJ,CAAA;AAlBY,oCAAY;uBAAZ,YAAY;IAFxB,IAAA,wCAAyB,GAAE;IAC3B,IAAA,sBAAU,GAAE;GACA,YAAY,CAkBxB","sourcesContent":["import { injectable } from 'inversify';\nimport {provideFrameworkSingleton, MethodMeta} from '@webpieces/http-routing';\nimport { Filter, WpResponse, Service } from '@webpieces/http-routing';\nimport { LogManager } from '@webpieces/core-util';\nimport { LogApiCall } from '@webpieces/core-util';\n\n/**\n * LogApiFilter - Structured API logging for all requests/responses.\n * Priority: 1800 (after ContextFilter at 2000, before custom filters)\n *\n * Logging patterns (via LogApiCall):\n * - [API-server-req] Class.method /url request={...}\n * - [API-server-resp-SUCCESS] Class.method response={...}\n * - [API-server-resp-FAIL] Class.method error=... (server errors: 500, 502, 504)\n * - [API-server-resp-OTHER] Class.method errorType=... (user errors: 400, 401, 403, 404, 266)\n *\n * Headers are read from RequestContext (NOT from meta.requestHeaders which is undefined\n * after ContextFilter runs at priority 2000).\n *\n * User errors (HttpBadRequestError, etc.) are logged as OTHER, not FAIL,\n * because they are expected behavior from the server's perspective.\n */\nconst log = LogManager.getLogger('LogApiFilter');\n\n@provideFrameworkSingleton()\n@injectable()\nexport class LogApiFilter extends Filter<MethodMeta, WpResponse<unknown>> {\n\n    async filter(\n        meta: MethodMeta,\n        nextFilter: Service<MethodMeta, WpResponse<unknown>>,\n    ): Promise<WpResponse<unknown>> {\n        // Wrap nextFilter.invoke in a method that returns the response\n        const method = async (): Promise<unknown> => {\n            const wpResponse = await nextFilter.invoke(meta);\n            return wpResponse.response;\n        };\n\n        // LogApiCall is a singleton (use it directly, no `new`). It logs the text lines AND stamps the\n        // structured `api={side:'server',...}` tag into RequestContext, so every log line during the\n        // request carries jsonPayload.api. Correlation fields (requestId, ...) are added by the backend.\n        const response = await LogApiCall.execute('server', meta.routeMeta, meta.requestDto, method);\n        return new WpResponse(response);\n    }\n}\n"]}