npm - koa-classic-server - Versions diffs - 2.0.0 → 2.1.2 - Mend

koa-classic-server 2.0.0 → 2.1.2

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (45) hide show

package/__tests__/publicWwwTest/ejs-templates/with-loop.ejs ADDED Viewed

@@ -0,0 +1,16 @@
+<!DOCTYPE html>
+<html lang="it">
+<head>
+    <meta charset="UTF-8">
+    <title>EJS Loop Test</title>
+</head>
+<body>
+    <h1>Lista di Items</h1>
+    <ul>
+    <% items.forEach(function(item, index) { %>
+        <li data-index="<%= index %>"><%= item.name %> - <%= item.value %></li>
+    <% }); %>
+    </ul>
+    <p>Totale items: <%= items.length %></p>
+</body>
+</html>

package/{scripts → __tests__}/setup-benchmark.js RENAMED Viewed

@@ -8,7 +8,7 @@
 const fs = require('fs');
 const path = require('path');
-const BENCHMARK_DIR = path.join(__dirname, '../benchmark-data');
+const BENCHMARK_DIR = path.join(__dirname, 'benchmark-data');
 console.log('🔧 Setting up benchmark environment...\n');

package/docs/CODE_REVIEW.md ADDED Viewed

@@ -0,0 +1,298 @@
+# Code Review - koa-classic-server
+## Analisi Generale del Codice
+Data: 2025-11-18
+File analizzato: `index.cjs` (651 righe)
+Test: 146 passing ✅
+---
+## 📊 Sommario
+| Categoria | Stato | Note |
+|-----------|-------|------|
+| Indentazione | ✅ Ottima | Consistente a 4 spazi |
+| Gestione errori | ✅ Ottima | Try/catch completi |
+| Sicurezza | ✅ Ottima | Path traversal, XSS protection |
+| Performance | ✅ Ottima | Async/await, array join |
+| Test coverage | ✅ Ottima | 146 test passano |
+| **Operatori confronto** | ⚠️ **Da migliorare** | Uso misto di `==` e `===` |
+---
+## ⚠️ Problemi Trovati
+### 1. Uso Inconsistente di == vs ===
+**Problema:** Il codice usa sia `==` che `===` in modo inconsistente.
+**Impatto:** Medio - Può causare bug sottili dovuti alla type coercion JavaScript.
+**Occorrenze trovate:** ~25 istanze
+#### Esempi:
+**Linea 68:**
+```javascript
+// ❌ Attuale
+options.showDirContents = typeof options.showDirContents == 'boolean' ? options.showDirContents : true;
+// ✅ Suggerito
+options.showDirContents = typeof options.showDirContents === 'boolean' ? options.showDirContents : true;
+```
+**Linea 112:**
+```javascript
+// ❌ Attuale
+if (ctx.href.charAt(ctx.href.length - 1) == '/') {
+// ✅ Suggerito
+if (ctx.href.charAt(ctx.href.length - 1) === '/') {
+```
+**Linea 123:**
+```javascript
+// ❌ Attuale
+if (a_urlPrefix[key] != a_pathname[key]) {
+// ✅ Suggerito
+if (a_urlPrefix[key] !== a_pathname[key]) {
+```
+**Linea 470:**
+```javascript
+// ❌ Attuale
+if (dir.length == 0) {
+// ✅ Suggerito
+if (dir.length === 0) {
+```
+**Linea 500:**
+```javascript
+// ❌ Attuale
+if (type == 1) {
+// ✅ Suggerito
+if (type === 1) {
+```
+---
+### 2. Lista Completa Sostituzioni Richieste
+#### typeof comparisons (5 occorrenze):
+```javascript
+Linea 68:  typeof options.showDirContents == 'boolean'  →  ===
+Linea 71:  typeof options.index == 'string'             →  ===
+Linea 94:  typeof options.urlPrefix == 'string'         →  ===
+Linea 96:  options.template.render == undefined         →  ===
+Linea 96:  typeof options.template.render == 'function' →  ===
+Linea 100: typeof options.cacheMaxAge == 'number'       →  ===
+Linea 101: typeof options.enableCaching == 'boolean'    →  ===
+```
+#### String comparisons (5 occorrenze):
+```javascript
+Linea 112: ctx.href.charAt(ctx.href.length - 1) == '/'  →  ===
+Linea 123: a_urlPrefix[key] != a_pathname[key]          →  !==
+Linea 131: options.urlPrefix != ""                       →  !==
+Linea 152: pageHrefOutPrefix.pathname == "/"             →  ===
+Linea 462: pageHrefOutPrefix.origin + "/" != pageHrefOutPrefix.href  →  !==
+Linea 489: pageHref.href == pageHref.origin + options.urlPrefix + "/"  →  ===
+```
+#### Number comparisons (8 occorrenze):
+```javascript
+Linea 142: a_pathnameOutPrefix[1] == value.substring(1) →  ===
+Linea 470: dir.length == 0                               →  ===
+Linea 482: type !== 1 && type !== 2 && type !== 3        →  ✅ già corretto
+Linea 500: type == 1                                     →  ===
+Linea 510: type == 2                                     →  ===
+Linea 511: type == 2 || type == 3                        →  === (2 volte)
+Linea 532: a.type === 2                                  →  ✅ già corretto
+Linea 534: a.type !== 2 && b.type === 2                  →  ✅ già corretto
+Linea 542: a.type === 2 && b.type !== 2                  →  ✅ già corretto
+Linea 557: item.type == 1                                →  ===
+```
+---
+## ✅ Punti di Forza
+### 1. Sicurezza
+- ✅ Path traversal protection completa (linea 149-168)
+- ✅ XSS protection con `escapeHtml()` (linea 524-534)
+- ✅ Content-Disposition properly quoted (linea 379-383)
+- ✅ Validazione input robusta
+### 2. Performance
+- ✅ Tutte le operazioni I/O sono async (non bloccanti)
+- ✅ String concatenation sostituita con array join
+- ✅ HTTP caching con ETag e Last-Modified
+- ✅ Conditional requests (304 Not Modified)
+### 3. Gestione Errori
+- ✅ Try/catch su tutte le operazioni async
+- ✅ Status code corretti (404, 403, 500)
+- ✅ Gestione race conditions
+- ✅ Fallback appropriati
+### 4. Codice Pulito
+- ✅ Indentazione consistente (4 spazi)
+- ✅ Commenti chiari e utili
+- ✅ Nomi variabili descrittivi
+- ✅ Funzioni ben separate
+---
+## 💡 Raccomandazioni
+### Priorità Alta
+1. **Standardizzare operatori di confronto**
+   - Sostituire tutti i `==` con `===`
+   - Sostituire tutti i `!=` con `!==`
+   - Motivo: Prevenire bug dovuti a type coercion
+   - Tempo stimato: 10-15 minuti
+   - Rischio: Basso (test coprono il comportamento)
+### Priorità Media
+2. **Aggiungere JSDoc comments**
+   ```javascript
+   /**
+    * Find index file in directory with priority support
+    * @param {string} dirPath - Directory path to search
+    * @param {Array<string|RegExp>} indexPatterns - Array of patterns
+    * @returns {Promise<{name: string, stat: fs.Stats}|null>}
+    */
+   async function findIndexFile(dirPath, indexPatterns) {
+       // ...
+   }
+   ```
+3. **Separare funzioni helper in moduli**
+   - `escapeHtml()` → `lib/htmlUtils.js`
+   - `formatSize()` → `lib/formatUtils.js`
+   - Miglior organizzazione e testabilità
+### Priorità Bassa
+4. **Aggiungere TypeScript definitions** (`index.d.ts`)
+5. **Considerare ESLint** per standardizzazione automatica
+---
+## 🧪 Verifica Sicurezza
+### Path Traversal Protection ✅
+```javascript
+// Linea 149-168
+const normalizedPath = path.normalize(requestedPath);
+const fullPath = path.join(normalizedRootDir, normalizedPath);
+// Security check: ensure resolved path is within rootDir
+if (!fullPath.startsWith(normalizedRootDir)) {
+    ctx.status = 403;
+    ctx.body = 'Forbidden';
+    return;
+}
+```
+**Valutazione:** Eccellente. Protegge da `../../../etc/passwd` attacks.
+### XSS Protection ✅
+```javascript
+// Linea 524-534
+function escapeHtml(unsafe) {
+    if (typeof unsafe !== 'string') {
+        return unsafe;
+    }
+    return unsafe
+        .replace(/&/g, "&amp;")
+        .replace(/</g, "&lt;")
+        .replace(/>/g, "&gt;")
+        .replace(/"/g, "&quot;")
+        .replace(/'/g, "&#039;");
+}
+```
+**Valutazione:** Eccellente. Previene XSS in directory listing.
+### Race Condition Protection ✅
+```javascript
+// Linea 174-181
+try {
+    stat = await fs.promises.stat(toOpen);
+} catch (error) {
+    ctx.status = 404;
+    ctx.body = requestedUrlNotFound();
+    return;
+}
+```
+**Valutazione:** Buona. Gestisce correttamente file cancellati tra check ed access.
+---
+## 📈 Metriche Codice
+| Metrica | Valore | Valutazione |
+|---------|--------|-------------|
+| Linee codice | 651 | ✅ Appropriato |
+| Complessità ciclomatica | Bassa | ✅ Ottimo |
+| Funzioni async | 100% | ✅ Eccellente |
+| Copertura test | Alta | ✅ Eccellente |
+| Dipendenze | 4 | ✅ Minime |
+---
+## 🎯 Piano d'Azione Suggerito
+### Step 1: Fix operatori confronto (15 min)
+```bash
+# Trova tutte le occorrenze
+grep -n " == " index.cjs
+grep -n " != " index.cjs
+# Sostituisci manualmente o con script
+```
+### Step 2: Verifica test (2 min)
+```bash
+npm test
+```
+### Step 3: Commit (1 min)
+```bash
+git add index.cjs
+git commit -m "Standardize comparison operators (== to ===, != to !==)"
+```
+---
+## 📝 Note Finali
+Il codice è **globalmente di ottima qualità**:
+- ✅ Sicuro
+- ✅ Performante
+- ✅ Ben testato
+- ✅ Ben strutturato
+L'unico miglioramento significativo è la **standardizzazione degli operatori di confronto** da `==` a `===`, che è una best practice JavaScript universalmente riconosciuta.
+**Rischio di modifiche:** Basso
+- I test esistenti coprono il comportamento
+- Le modifiche sono meccaniche
+- Nessun cambio di logica
+**Benefici:**
+- Codice più robusto
+- Prevenzione bug futuri
+- Conformità best practices
+- Migliore leggibilità
+---
+**Review by:** Claude Code Assistant
+**Date:** 2025-11-18
+**Status:** ⚠️ Minor issues found - Easy fixes recommended