@camcima/nestjs-rfc9457 0.2.0 → 0.3.0

package/README.md

@@ -77,6 +77,7 @@ Extension members (arbitrary key-value pairs) are allowed for problem-type-speci
 - Four `instance` strategies: `'request-uri'`, `'uuid'`, `'none'`, or a custom callback
 - Optional catch-all mode for non-`HttpException` throwables (produces 500 Problem Details)
 - Custom `exceptionMapper` callback for full control over any exception
+- Default `error`-level logging of unhandled exceptions when `catchAllExceptions: true` (override via `onUnhandled` callback)
 - `ProblemDetailsFactory` is injectable — use it directly in GraphQL, microservices, or custom filters
 - Optional `@nestjs/swagger` integration: `ProblemDetailDto` and `ValidationProblemDetailDto` for OpenAPI documentation, plus a `applyProblemDetailResponses()` helper that auto-applies `@ApiResponse` decorators to all controllers under `application/problem+json`
 - Works with both Express and Fastify adapters
@@ -260,6 +261,8 @@ When `false` (default), exceptions that are not `HttpException` instances are pa
 Rfc9457Module.forRoot({ catchAllExceptions: true });
 ```
 
+**Observability:** when this branch fires (a non-`HttpException` reaches the filter and no `exceptionMapper` claims it), the library logs the exception at `error` level via NestJS's built-in `Logger` (context `Rfc9457ExceptionFilter`) before sending the generic 500. This keeps unexpected throwables visible in server logs even though the response body is intentionally bland. To redirect or replace this logging, use the [`onUnhandled`](#onunhandled) callback described below.
+
 ### `exceptionMapper`
 
 **Type**: `(exception: unknown, request: Rfc9457Request) => ProblemDetail | null`
@@ -284,6 +287,28 @@ Rfc9457Module.forRoot({
 
 If the returned `ProblemDetail` omits `status`, the factory falls back to `exception.getStatus()` (if it is an `HttpException`) or `500`.
 
+### `onUnhandled`
+
+**Type**: `(exception: unknown, request: Rfc9457Request) => void` | **Default**: built-in `Logger.error(...)` (context `Rfc9457ExceptionFilter`)
+
+Called when a non-`HttpException` reaches the catch-all branch (i.e. `catchAllExceptions: true` AND the `exceptionMapper` returned `null`). Use this to send unhandled exceptions to a structured sink (Sentry, Datadog, a custom pino child logger) or to suppress the default log entirely.
+
+```typescript
+Rfc9457Module.forRoot({
+  catchAllExceptions: true,
+  onUnhandled: (exception, request) => {
+    // Route to Sentry, Datadog, etc.
+    sentry.captureException(exception, {
+      tags: { method: request.method, url: request.url },
+    });
+  },
+});
+```
+
+**The filter still sends the generic 500 Problem Details response after invoking `onUnhandled`.** This callback exists purely for observability — it never changes the HTTP response.
+
+When `onUnhandled` is **not** provided, the library calls `Logger.error(...)` with either the exception's `stack` string or a `{ exception }` structured context (for non-`Error` values). The log context is `Rfc9457ExceptionFilter` so it can be filtered or silenced via NestJS's logger configuration.
+
 ### `validationExceptionMapper`
 
 **Type**: `(messages: string[], request: Rfc9457Request) => ProblemDetail`
@@ -930,13 +955,13 @@ Gitleaks must be [installed locally](https://github.com/gitleaks/gitleaks#instal
 
 ```bash
 # Dependency audit (production only)
-npm run audit:deps
+pnpm run audit:deps
 
 # Secret scanning
-npm run audit:secrets
+pnpm run audit:secrets
 
-# Full npm audit (all dependencies)
-npm audit
+# Full pnpm audit (all dependencies)
+pnpm audit
 ```
 
 ---
@@ -951,19 +976,19 @@ git clone https://github.com/camcima/nestjs-rfc9457.git
 cd nestjs-rfc9457
 
 # Install dependencies
-npm install
+pnpm install
 
 # Run unit tests
-npm run test:unit
+pnpm run test:unit
 
 # Run e2e tests
-npm run test:e2e
+pnpm run test:e2e
 
 # Run all tests with coverage
-npm run test:cov
+pnpm run test:cov
 
 # Build
-npm run build
+pnpm run build
 ```
 
 This project uses [Conventional Commits](https://www.conventionalcommits.org/) enforced by commitlint, and [Lefthook](https://github.com/evilmartians/lefthook) for pre-commit hooks (lint + format on staged files).

package/dist/rfc9457.exception-filter.d.ts

@@ -6,6 +6,7 @@ export declare class Rfc9457ExceptionFilter extends BaseExceptionFilter {
     private readonly factory;
     private readonly options;
     private readonly adapterHost;
+    private readonly logger;
     constructor(factory: ProblemDetailsFactory, options: Rfc9457ModuleOptions, adapterHost: HttpAdapterHost);
     catch(exception: unknown, host: ArgumentsHost): void;
 }

package/dist/rfc9457.exception-filter.js

@@ -11,18 +11,20 @@ var __metadata = (this && this.__metadata) || function (k, v) {
 var __param = (this && this.__param) || function (paramIndex, decorator) {
     return function (target, key) { decorator(target, key, paramIndex); }
 };
+var Rfc9457ExceptionFilter_1;
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.Rfc9457ExceptionFilter = void 0;
 const common_1 = require("@nestjs/common");
 const core_1 = require("@nestjs/core");
 const problem_details_factory_1 = require("./problem-details.factory");
 const rfc9457_constants_1 = require("./rfc9457.constants");
-let Rfc9457ExceptionFilter = class Rfc9457ExceptionFilter extends core_1.BaseExceptionFilter {
+let Rfc9457ExceptionFilter = Rfc9457ExceptionFilter_1 = class Rfc9457ExceptionFilter extends core_1.BaseExceptionFilter {
     constructor(factory, options, adapterHost) {
         super(adapterHost.httpAdapter);
         this.factory = factory;
         this.options = options;
         this.adapterHost = adapterHost;
+        this.logger = new common_1.Logger(Rfc9457ExceptionFilter_1.name);
     }
     catch(exception, host) {
         if (host.getType() !== 'http') {
@@ -47,6 +49,18 @@ let Rfc9457ExceptionFilter = class Rfc9457ExceptionFilter extends core_1.BaseExc
             super.catch(exception, host);
             return;
         }
+        if (!isHttpException) {
+            if (this.options.onUnhandled) {
+                const ctx = host.switchToHttp();
+                this.options.onUnhandled(exception, ctx.getRequest());
+            }
+            else if (exception instanceof Error) {
+                this.logger.error(`Unhandled non-HTTP exception: ${exception.message}`, exception.stack);
+            }
+            else {
+                this.logger.error('Unhandled non-HTTP exception (non-Error value thrown)', exception);
+            }
+        }
         const ctx = host.switchToHttp();
         const request = ctx.getRequest();
         const response = ctx.getResponse();
@@ -57,7 +71,7 @@ let Rfc9457ExceptionFilter = class Rfc9457ExceptionFilter extends core_1.BaseExc
     }
 };
 exports.Rfc9457ExceptionFilter = Rfc9457ExceptionFilter;
-exports.Rfc9457ExceptionFilter = Rfc9457ExceptionFilter = __decorate([
+exports.Rfc9457ExceptionFilter = Rfc9457ExceptionFilter = Rfc9457ExceptionFilter_1 = __decorate([
     (0, common_1.Catch)(),
     __param(1, (0, common_1.Inject)(rfc9457_constants_1.RFC9457_MODULE_OPTIONS)),
     __metadata("design:paramtypes", [problem_details_factory_1.ProblemDetailsFactory, Object, core_1.HttpAdapterHost])

package/dist/rfc9457.exception-filter.js.map

@@ -1 +1 @@
-{"version":3,"file":"rfc9457.exception-filter.js","sourceRoot":"","sources":["../src/rfc9457.exception-filter.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;AAAA,2CAA6E;AAC7E,uCAAoE;AACpE,uEAAkE;AAClE,2DAAmF;AAI5E,IAAM,sBAAsB,GAA5B,MAAM,sBAAuB,SAAQ,0BAAmB;IAC7D,YACmB,OAA8B,EACE,OAA6B,EAC7D,WAA4B;QAE7C,KAAK,CAAC,WAAW,CAAC,WAAW,CAAC,CAAC;QAJd,YAAO,GAAP,OAAO,CAAuB;QACE,YAAO,GAAP,OAAO,CAAsB;QAC7D,gBAAW,GAAX,WAAW,CAAiB;IAG/C,CAAC;IAED,KAAK,CAAC,SAAkB,EAAE,IAAmB;QAE3C,IAAI,IAAI,CAAC,OAAO,EAAE,KAAK,MAAM,EAAE,CAAC;YAC9B,KAAK,CAAC,KAAK,CAAC,SAAS,EAAE,IAAI,CAAC,CAAC;YAC7B,OAAO;QACT,CAAC;QAED,MAAM,eAAe,GAAG,SAAS,YAAY,sBAAa,CAAC;QAK3D,IAAI,IAAI,CAAC,OAAO,CAAC,eAAe,EAAE,CAAC;YACjC,MAAM,GAAG,GAAG,IAAI,CAAC,YAAY,EAAE,CAAC;YAChC,MAAM,OAAO,GAAG,GAAG,CAAC,UAAU,EAAE,CAAC;YACjC,MAAM,MAAM,GAAG,IAAI,CAAC,OAAO,CAAC,eAAe,CAAC,SAAS,EAAE,OAAO,CAAC,CAAC;YAChE,IAAI,MAAM,EAAE,CAAC;gBACX,MAAM,QAAQ,GAAG,GAAG,CAAC,WAAW,EAAE,CAAC;gBACnC,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,GAAG,IAAI,CAAC,OAAO,CAAC,gBAAgB,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,CAAC,CAAC;gBACnF,MAAM,WAAW,GAAG,IAAI,CAAC,WAAW,CAAC,WAAW,CAAC;gBACjD,WAAW,CAAC,SAAS,CAAC,QAAQ,EAAE,cAAc,EAAE,wCAAoB,CAAC,CAAC;gBACtE,WAAW,CAAC,KAAK,CAAC,QAAQ,EAAE,IAAI,EAAE,MAAM,CAAC,CAAC;gBAC1C,OAAO;YACT,CAAC;QACH,CAAC;QAGD,IAAI,CAAC,eAAe,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,kBAAkB,EAAE,CAAC;YACzD,KAAK,CAAC,KAAK,CAAC,SAAS,EAAE,IAAI,CAAC,CAAC;YAC7B,OAAO;QACT,CAAC;QAED,MAAM,GAAG,GAAG,IAAI,CAAC,YAAY,EAAE,CAAC;QAChC,MAAM,OAAO,GAAG,GAAG,CAAC,UAAU,EAAE,CAAC;QACjC,MAAM,QAAQ,GAAG,GAAG,CAAC,WAAW,EAAE,CAAC;QAEnC,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,GAAG,IAAI,CAAC,OAAO,CAAC,MAAM,CAAC,SAAS,EAAE,OAAO,EAAE,EAAE,UAAU,EAAE,IAAI,EAAE,CAAC,CAAC;QACvF,MAAM,WAAW,GAAG,IAAI,CAAC,WAAW,CAAC,WAAW,CAAC;QACjD,WAAW,CAAC,SAAS,CAAC,QAAQ,EAAE,cAAc,EAAE,wCAAoB,CAAC,CAAC;QACtE,WAAW,CAAC,KAAK,CAAC,QAAQ,EAAE,IAAI,EAAE,MAAM,CAAC,CAAC;IAC5C,CAAC;CACF,CAAA;AAlDY,wDAAsB;iCAAtB,sBAAsB;IADlC,IAAA,cAAK,GAAE;IAIH,WAAA,IAAA,eAAM,EAAC,0CAAsB,CAAC,CAAA;qCADL,+CAAqB,UAEjB,sBAAe;GAJpC,sBAAsB,CAkDlC"}
+{"version":3,"file":"rfc9457.exception-filter.js","sourceRoot":"","sources":["../src/rfc9457.exception-filter.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;AAAA,2CAAqF;AACrF,uCAAoE;AACpE,uEAAkE;AAClE,2DAAmF;AAI5E,IAAM,sBAAsB,8BAA5B,MAAM,sBAAuB,SAAQ,0BAAmB;IAG7D,YACmB,OAA8B,EACf,OAA8C,EAC7D,WAA4B;QAE7C,KAAK,CAAC,WAAW,CAAC,WAAW,CAAC,CAAC;QAJd,YAAO,GAAP,OAAO,CAAuB;QACE,YAAO,GAAP,OAAO,CAAsB;QAC7D,gBAAW,GAAX,WAAW,CAAiB;QAL9B,WAAM,GAAG,IAAI,eAAM,CAAC,wBAAsB,CAAC,IAAI,CAAC,CAAC;IAQlE,CAAC;IAED,KAAK,CAAC,SAAkB,EAAE,IAAmB;QAE3C,IAAI,IAAI,CAAC,OAAO,EAAE,KAAK,MAAM,EAAE,CAAC;YAC9B,KAAK,CAAC,KAAK,CAAC,SAAS,EAAE,IAAI,CAAC,CAAC;YAC7B,OAAO;QACT,CAAC;QAED,MAAM,eAAe,GAAG,SAAS,YAAY,sBAAa,CAAC;QAK3D,IAAI,IAAI,CAAC,OAAO,CAAC,eAAe,EAAE,CAAC;YACjC,MAAM,GAAG,GAAG,IAAI,CAAC,YAAY,EAAE,CAAC;YAChC,MAAM,OAAO,GAAG,GAAG,CAAC,UAAU,EAAE,CAAC;YACjC,MAAM,MAAM,GAAG,IAAI,CAAC,OAAO,CAAC,eAAe,CAAC,SAAS,EAAE,OAAO,CAAC,CAAC;YAChE,IAAI,MAAM,EAAE,CAAC;gBACX,MAAM,QAAQ,GAAG,GAAG,CAAC,WAAW,EAAE,CAAC;gBACnC,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,GAAG,IAAI,CAAC,OAAO,CAAC,gBAAgB,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,CAAC,CAAC;gBACnF,MAAM,WAAW,GAAG,IAAI,CAAC,WAAW,CAAC,WAAW,CAAC;gBACjD,WAAW,CAAC,SAAS,CAAC,QAAQ,EAAE,cAAc,EAAE,wCAAoB,CAAC,CAAC;gBACtE,WAAW,CAAC,KAAK,CAAC,QAAQ,EAAE,IAAI,EAAE,MAAM,CAAC,CAAC;gBAC1C,OAAO;YACT,CAAC;QACH,CAAC;QAGD,IAAI,CAAC,eAAe,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,kBAAkB,EAAE,CAAC;YACzD,KAAK,CAAC,KAAK,CAAC,SAAS,EAAE,IAAI,CAAC,CAAC;YAC7B,OAAO;QACT,CAAC;QAYD,IAAI,CAAC,eAAe,EAAE,CAAC;YACrB,IAAI,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,CAAC;gBAC7B,MAAM,GAAG,GAAG,IAAI,CAAC,YAAY,EAAE,CAAC;gBAChC,IAAI,CAAC,OAAO,CAAC,WAAW,CAAC,SAAS,EAAE,GAAG,CAAC,UAAU,EAAE,CAAC,CAAC;YACxD,CAAC;iBAAM,IAAI,SAAS,YAAY,KAAK,EAAE,CAAC;gBAGtC,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,iCAAiC,SAAS,CAAC,OAAO,EAAE,EAAE,SAAS,CAAC,KAAK,CAAC,CAAC;YAC3F,CAAC;iBAAM,CAAC;gBACN,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,uDAAuD,EAAE,SAAS,CAAC,CAAC;YACxF,CAAC;QACH,CAAC;QAED,MAAM,GAAG,GAAG,IAAI,CAAC,YAAY,EAAE,CAAC;QAChC,MAAM,OAAO,GAAG,GAAG,CAAC,UAAU,EAAE,CAAC;QACjC,MAAM,QAAQ,GAAG,GAAG,CAAC,WAAW,EAAE,CAAC;QAEnC,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,GAAG,IAAI,CAAC,OAAO,CAAC,MAAM,CAAC,SAAS,EAAE,OAAO,EAAE,EAAE,UAAU,EAAE,IAAI,EAAE,CAAC,CAAC;QACvF,MAAM,WAAW,GAAG,IAAI,CAAC,WAAW,CAAC,WAAW,CAAC;QACjD,WAAW,CAAC,SAAS,CAAC,QAAQ,EAAE,cAAc,EAAE,wCAAoB,CAAC,CAAC;QACtE,WAAW,CAAC,KAAK,CAAC,QAAQ,EAAE,IAAI,EAAE,MAAM,CAAC,CAAC;IAC5C,CAAC;CACF,CAAA;AA3EY,wDAAsB;iCAAtB,sBAAsB;IADlC,IAAA,cAAK,GAAE;IAMH,WAAA,IAAA,eAAM,EAAC,0CAAsB,CAAC,CAAA;qCADL,+CAAqB,UAEjB,sBAAe;GANpC,sBAAsB,CA2ElC"}

package/dist/rfc9457.interfaces.d.ts

@@ -24,6 +24,7 @@ export interface Rfc9457ModuleOptions {
     catchAllExceptions?: boolean;
     exceptionMapper?: (exception: unknown, request: Rfc9457Request) => ProblemDetail | null;
     validationExceptionMapper?: (messages: string[], request: Rfc9457Request) => ProblemDetail;
+    onUnhandled?: (exception: unknown, request: Rfc9457Request) => void;
 }
 export interface Rfc9457OptionsFactory {
     createRfc9457Options(): Promise<Rfc9457ModuleOptions> | Rfc9457ModuleOptions;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@camcima/nestjs-rfc9457",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "NestJS library for RFC 9457 Problem Details responses",
   "author": "Carlos Cima",
   "license": "MIT",
@@ -22,6 +22,7 @@
   "engines": {
     "node": ">=18"
   },
+  "packageManager": "pnpm@9.15.0",
   "scripts": {
     "build": "tsc -p tsconfig.build.json",
     "lint": "eslint 'src/**/*.ts' 'test/**/*.ts'",
@@ -31,17 +32,17 @@
     "test:unit": "jest --testPathPattern=test/unit",
     "test:e2e": "jest --testPathPattern=test/e2e",
     "test:cov": "jest --coverage",
-    "prepublishOnly": "npm run build",
+    "prepublishOnly": "pnpm run build",
     "release": "release-it",
-    "audit:deps": "npm audit --omit=dev",
+    "audit:deps": "pnpm audit --prod",
     "audit:secrets": "gitleaks git --no-banner --redact -v"
   },
   "peerDependencies": {
     "@nestjs/common": "^10.0.0 || ^11.0.0",
     "@nestjs/core": "^10.0.0 || ^11.0.0",
+    "@nestjs/swagger": "^7.0.0 || ^8.0.0 || ^11.0.0",
     "class-validator": "^0.14.0",
-    "reflect-metadata": "^0.1.13 || ^0.2.0",
-    "@nestjs/swagger": "^7.0.0 || ^8.0.0 || ^11.0.0"
+    "reflect-metadata": "^0.1.13 || ^0.2.0"
   },
   "peerDependenciesMeta": {
     "class-validator": {
@@ -54,14 +55,14 @@
   "devDependencies": {
     "@commitlint/cli": "^20.5.0",
     "@commitlint/config-conventional": "^20.5.0",
-    "@nestjs/common": "^11.1.18",
-    "@nestjs/core": "^11.1.18",
-    "@nestjs/swagger": "^11.1.18",
-    "@nestjs/platform-express": "^11.1.18",
-    "@nestjs/platform-fastify": "^11.1.18",
-    "@nestjs/testing": "^11.1.18",
+    "@nestjs/common": "^11.1.19",
+    "@nestjs/core": "^11.1.19",
+    "@nestjs/platform-express": "^11.1.19",
+    "@nestjs/platform-fastify": "^11.1.19",
+    "@nestjs/swagger": "^11.2.7",
+    "@nestjs/testing": "^11.1.19",
     "@types/jest": "^30.0.0",
-    "@types/node": "^22.19.17",
+    "@types/node": "^25.6.0",
     "@types/supertest": "^7.2.0",
     "@typescript-eslint/eslint-plugin": "^7.0.0",
     "@typescript-eslint/parser": "^7.0.0",
@@ -70,10 +71,9 @@
     "eslint": "^8.0.0",
     "eslint-config-prettier": "^9.0.0",
     "eslint-plugin-prettier": "^5.0.0",
-    "fastify": "^4.0.0",
     "jest": "^30.3.0",
     "lefthook": "^1.0.0",
-    "prettier": "^3.0.0",
+    "prettier": "^3.8.3",
     "reflect-metadata": "^0.2.0",
     "release-it": "^18.0.0",
     "rxjs": "^7.0.0",
@@ -82,6 +82,15 @@
     "ts-node": "^10.9.2",
     "typescript": "^6.0.2"
   },
+  "pnpm": {
+    "overrides": {
+      "basic-ftp@<5.3.0": ">=5.3.0",
+      "fastify@<5.8.5": ">=5.8.5",
+      "lodash@<4.18.0": ">=4.18.0",
+      "path-to-regexp@<8.4.0": ">=8.4.0",
+      "undici@<6.24.0": ">=6.24.0"
+    }
+  },
   "repository": {
     "type": "git",
     "url": "git+https://github.com/camcima/nestjs-rfc9457.git"