@liberstudio/cloudflare-list 3.0.0 → 3.0.3

package/README.md

@@ -2,12 +2,12 @@
 
 Modulo NestJS per il rilevamento e blocco automatico di attacchi tramite Cloudflare IP List.
 
-Ogni richiesta che produce un `404` viene considerata sospetta (tipicamente bot, scanner, enumeratori). L'IP viene risolto tramite ASN lookup per ottenere l'intero range di subnet e bloccarlo direttamente sulla lista Cloudflare WAF. Se il lookup fallisce, viene bloccato il singolo IP.
+Ogni richiesta che produce un `404` su un indirizzo senza nessuna rotta viene considerata sospetta (tipicamente bot, scanner, enumeratori: `/wp-admin`, `/.env`). Il `404` restituito da una rotta esistente (es. `GET /artworks/:id` di una risorsa che non c'è) non viene bloccato. L'IP viene risolto tramite ASN lookup per ottenere l'intero range di subnet e bloccarlo direttamente sulla lista Cloudflare WAF. Se il lookup fallisce, viene bloccato il singolo IP.
 
 ## Come funziona
 
 ```
-Request → 404 rilevato dal middleware
+Request → 404 rilevato dal middleware (solo se nessuna rotta ha risposto)
   └─ IP estratto (cf-connecting-ip / x-real-ip / x-forwarded-for)
        └─ throttle 5s per IP
             └─ log su file (se path non è silentPath)

package/dist/middleware/attack-logger.middleware.d.ts

@@ -15,6 +15,7 @@ export declare class AttackLoggerMiddleware implements NestMiddleware, OnModuleD
     private isThrottled;
     private sanitize;
     private ensureDirectoryExists;
+    private hasRealRoute;
     private isExcluded;
     private isSilent;
     onModuleDestroy(): void;

package/dist/middleware/attack-logger.middleware.js

@@ -42,7 +42,7 @@ let AttackLoggerMiddleware = AttackLoggerMiddleware_1 = class AttackLoggerMiddle
     }
     use(req, res, next) {
         res.on("finish", () => {
-            if (res.statusCode === 404 && !this.isExcluded(req.url)) {
+            if (res.statusCode === 404 && !this.hasRealRoute(req) && !this.isExcluded(req.url)) {
                 this.handleSuspicious(req);
             }
         });
@@ -85,7 +85,7 @@ let AttackLoggerMiddleware = AttackLoggerMiddleware_1 = class AttackLoggerMiddle
             this.logger.warn(`IP non valido: ${ip}`);
             return;
         }
-        this.logger.warn(`Provenienza ${ip}: socket=${origin.remoteAddress} host=${origin.host} cf-ray=${origin.cfRay}`);
+        this.logger.warn(`Provenienza ${ip}: ${req.method} ${this.sanitize(req.originalUrl)} → 404, socket=${origin.remoteAddress} host=${origin.host} cf-ray=${origin.cfRay}`);
         this.attSrv.blockIp(ip).catch((err) => {
             const msg = err instanceof Error ? err.message : "Errore sconosciuto";
             this.logger.error(`Aggiornamento Cloudflare fallito: ${msg}`);
@@ -110,6 +110,13 @@ let AttackLoggerMiddleware = AttackLoggerMiddleware_1 = class AttackLoggerMiddle
             this.logger.log(`Cartella dei log creata: ${dir}`);
         }
     }
+    // Un 404 di una rotta esistente (es. risorsa non trovata) non è una scansione.
+    // Express lascia req.route solo se una rotta ha risposto; Nest 12 con prefisso globale
+    // gestisce gli indirizzi senza rotta con il catch-all '*path', che quindi non conta.
+    hasRealRoute(req) {
+        const path = req.route?.path;
+        return path !== undefined && path !== "*path";
+    }
     isExcluded(url) {
         const excluded = this.options.excludedPaths ?? [];
         return excluded.some((p) => (p instanceof RegExp ? p.test(url) : url.startsWith(p)));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@liberstudio/cloudflare-list",
-  "version": "3.0.0",
+  "version": "3.0.3",
   "description": "Modulo NestJS per gestione IP List Cloudflare",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -14,6 +14,7 @@
   ],
   "scripts": {
     "build": "npx tsc",
+    "test": "vitest run",
     "prepublishOnly": "npm run build",
     "version:bump": "npx tsx src/scripts/version-bump.ts",
     "pub": "npm run build && npm publish --access public"
@@ -35,12 +36,14 @@
     "@nestjs/axios": "^12.0.1",
     "@nestjs/common": "^12.4.0",
     "@nestjs/core": "^12.4.0",
+    "@nestjs/platform-express": "^12.4.2",
     "@types/express": "^5.0.6",
-    "@types/node": "^20.19.43",
+    "@types/node": "^24.19.2",
     "axios": "^1.21.0",
     "rxjs": "^7.8.2",
     "simple-git-hooks": "^2.13.1",
-    "typescript": "^5.0.0"
+    "typescript": "^5.0.0",
+    "vitest": "^5.1.0"
   },
   "allowScripts": {
     "simple-git-hooks@2.13.1": true