@ndiinginc/router 4.6.0 → 4.6.2

package/README.md

@@ -51,19 +51,23 @@ app.listen(3000);
 
 Yang diekspor adalah class `App`, dengan tiga properti tambahan:
 
-| Export       | Keterangan                               |
-| ------------ | ---------------------------------------- |
-| `App`        | Router + HTTP server (`listen`)          |
+| Export | Keterangan |
+| --- | --- |
+| `App` | Router + HTTP server (`listen`) |
 | `App.Router` | Router tanpa server, untuk nested router |
-| `App.json`   | Middleware JSON body parser              |
-| `App.cors`   | Middleware CORS                          |
+| `App.json` | Middleware JSON body parser |
+| `App.cors` | Middleware CORS |
 
 ## Routing
 
 Semua method mengembalikan `this`, jadi bisa di-chain.
 
 ```js
-app.get("/users", handler).post("/users", handler).put("/users/:id", handler).patch("/users/:id", handler).delete("/users/:id", handler);
+app.get("/users", handler)
+   .post("/users", handler)
+   .put("/users/:id", handler)
+   .patch("/users/:id", handler)
+   .delete("/users/:id", handler);
 ```
 
 Method yang tersedia: `get`, `post`, `put`, `patch`, `delete`, `head`, `options`, `trace`, `connect`, `query`, dan `use` (semua method).
@@ -194,18 +198,18 @@ Jika tidak ada handler yang mengirim response, router mengirim JSON otomatis:
 
 ## Timeout
 
-Aktifkan lewat opsi `timeout` (milidetik) pada `App`. Jika seluruh chain handler melebihi waktu tersebut, error `504 Gateway Timeout` dilempar dan diteruskan ke error handler.
+Aktifkan lewat opsi `requestTimeout` (milidetik) pada `App`. Jika seluruh chain handler melebihi waktu tersebut, error `504 Gateway Timeout` dilempar dan diteruskan ke error handler.
 
 ```js
-const app = new App({ timeout: 5000 });
+const app = new App({ requestTimeout: 5000 });
 ```
 
-- Tanpa opsi `timeout` (atau bernilai `0`), tidak ada batas waktu
+- Tanpa opsi `requestTimeout` (atau bernilai `0`), tidak ada batas waktu
 - Handler yang sudah berjalan tidak dihentikan paksa. Tetapi setelah response dikirim, `res.json()` dan `res.send()` mengabaikan penulisan berikutnya, dan handler selanjutnya di chain tidak dipanggil
 
 ### `req.signal`
 
-Saat `timeout` aktif, `req.signal` adalah `AbortSignal` yang di-abort ketika timeout terjadi (reason-nya error 504). Teruskan ke operasi yang mendukungnya agar pekerjaan ikut dibatalkan:
+Saat `requestTimeout` aktif, `req.signal` adalah `AbortSignal` yang di-abort ketika timeout terjadi (reason-nya error 504) atau ketika salah satu handler melempar error (reason-nya error tersebut). Teruskan ke operasi yang mendukungnya agar pekerjaan ikut dibatalkan:
 
 ```js
 app.get("/proxy", async (req, res) => {
@@ -214,17 +218,17 @@ app.get("/proxy", async (req, res) => {
 });
 ```
 
-`req.signal` bernilai `undefined` jika `timeout` tidak diaktifkan.
+`req.signal` bernilai `undefined` jika `requestTimeout` tidak diaktifkan.
 
 ## Request
 
-| Properti     | Keterangan                                                                 |
-| ------------ | -------------------------------------------------------------------------- |
-| `req.path`   | Pathname tanpa query string                                                |
-| `req.query`  | Object query string, di-parse secara lazy. Key yang berulang menjadi array |
-| `req.params` | Parameter dari route dinamis (sudah di-decode)                             |
-| `req.body`   | Hasil parse body (butuh middleware `json()`)                               |
-| `req.signal` | `AbortSignal` yang di-abort saat timeout (hanya jika `timeout` aktif)      |
+| Properti | Keterangan |
+| --- | --- |
+| `req.path` | Pathname tanpa query string |
+| `req.query` | Object query string, di-parse secara lazy. Key yang berulang menjadi array, dan `+` di-decode menjadi spasi |
+| `req.params` | Parameter dari route dinamis (sudah di-decode) |
+| `req.body` | Hasil parse body (butuh middleware `json()`) |
+| `req.signal` | `AbortSignal` yang di-abort saat timeout atau error handler (hanya jika `requestTimeout` aktif) |
 
 ```js
 // GET /search?tag=a&tag=b&page=2
@@ -239,11 +243,11 @@ app.get("/search", (req, res) => {
 
 ## Response
 
-| Method             | Keterangan                                                  |
-| ------------------ | ----------------------------------------------------------- |
+| Method | Keterangan |
+| --- | --- |
 | `res.status(code)` | Set status code, mengembalikan `res` sehingga bisa di-chain |
-| `res.send(chunk)`  | Kirim response dan akhiri                                   |
-| `res.json(value)`  | Set `Content-Type: application/json`, serialize, lalu kirim |
+| `res.send(chunk)` | Kirim response dan akhiri |
+| `res.json(value)` | Set `Content-Type: application/json`, serialize, lalu kirim |
 
 ```js
 res.status(201).json({ created: true });
@@ -259,8 +263,8 @@ res.status(201).json({ created: true });
 app.use(App.json({ limit: 100 * 1024 })); // default 100 KB
 ```
 
-| Opsi    | Default  | Keterangan                      |
-| ------- | -------- | ------------------------------- |
+| Opsi | Default | Keterangan |
+| --- | --- | --- |
 | `limit` | `102400` | Ukuran body maksimum dalam byte |
 
 Perilaku:
@@ -285,19 +289,19 @@ app.use(
 );
 ```
 
-| Opsi          | Default | Keterangan                                                     |
-| ------------- | ------- | -------------------------------------------------------------- |
-| `origin`      | `"*"`   | `"*"`, string origin, atau `null`                              |
-| `credentials` | `false` | Mengirim `Access-Control-Allow-Credentials: true`              |
-| `headers`     | -       | `"*"` atau array nama header                                   |
-| `methods`     | -       | `"*"` atau array nama method                                   |
-| `maxAge`      | `0`     | Lama cache preflight (detik). `0` berarti header tidak dikirim |
+| Opsi | Default | Keterangan |
+| --- | --- | --- |
+| `origin` | `"*"` | `"*"`, satu origin, daftar dipisah koma, pola dengan `*` (misalnya `https://*.example.com`), atau `null` |
+| `credentials` | `false` | Mengirim `Access-Control-Allow-Credentials: true` |
+| `headers` | - | `"*"` atau array nama header |
+| `methods` | - | `"*"` atau array nama method |
+| `maxAge` | `0` | Lama cache preflight (detik). `0` berarti header tidak dikirim |
 
 Perilaku:
 
 - Request `OPTIONS` dijawab langsung dengan `204` dan chain berhenti
 - `Access-Control-Allow-Headers` dan `Access-Control-Allow-Methods` hanya dikirim pada preflight, jika nilai yang diminta ada di daftar
-- Dengan `origin: "*"` dan `credentials: true`, origin dari request dipantulkan dan header `Vary: Origin` ditambahkan
+- Jika `origin` berupa daftar atau pola (atau `"*"` dengan `credentials: true`), origin dari request dipantulkan bila cocok. Pencocokan dilakukan pada seluruh string (tanpa memperhatikan huruf besar/kecil), dan header `Vary: Origin` ditambahkan pada request non-preflight
 - Nilai `"*"` untuk `headers` dan `methods` diabaikan saat `credentials: true`, sesuai spesifikasi CORS
 - Pada request non-preflight, `headers` (jika array) dikirim sebagai `Access-Control-Expose-Headers`
 
@@ -306,21 +310,21 @@ Perilaku:
 ## Opsi
 
 ```js
-const app = new App({ size: 1000, timeout: 5000 });
+const app = new App({ cacheSize: 1000, requestTimeout: 5000 });
 ```
 
-| Opsi      | Default | Keterangan                                                                    |
-| --------- | ------- | ----------------------------------------------------------------------------- |
-| `size`    | `1000`  | Jumlah maksimum entri LRU cache untuk hasil lookup `METHOD path`              |
-| `timeout` | -       | Batas waktu handler dalam ms (khusus `App`). Lihat bagian [Timeout](#timeout) |
+| Opsi | Default | Keterangan |
+| --- | --- | --- |
+| `cacheSize` | `1000` | Jumlah maksimum entri LRU cache untuk hasil lookup `METHOD path` |
+| `requestTimeout` | - | Batas waktu handler dalam ms (khusus `App`). Lihat bagian [Timeout](#requestTimeout) |
 
-`Router` hanya menerima `size`.
+`Router` hanya menerima `cacheSize`.
 
 ## API
 
 ### `new App([options])` / `new Router([options])`
 
-Membuat instance baru. `App` adalah `Router` ditambah `listen()` dan dukungan `timeout`.
+Membuat instance baru. `App` adalah `Router` ditambah `listen()` dan dukungan `requestTimeout`.
 
 ### `router.use([path], ...handlers | router)`
 
@@ -351,11 +355,11 @@ console.log(server.address().port);
 
 Hello world dengan satu route `GET /`, diuji memakai `autocannon -c 100 -d 40 -p 10` di mesin lokal.
 
-|                       | Req/sec (avg) | Latency (avg) |
-| --------------------- | ------------- | ------------- |
-| Fastify               | 31.475        | 31,27 ms      |
-| **@ndiinginc/router** | 29.735        | 33,12 ms      |
-| Express               | 20.211        | 48,93 ms      |
+| | Req/sec (avg) | Latency (avg) |
+| --- | --- | --- |
+| Fastify | 31.475 | 31,27 ms |
+| **@ndiinginc/router** | 29.735 | 33,12 ms |
+| Express | 20.211 | 48,93 ms |
 
 Hasil bervariasi tergantung mesin, versi Node.js, dan jumlah route. Benchmark ini hanya menggambarkan overhead dasar, bukan performa aplikasi nyata.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ndiinginc/router",
-  "version": "4.6.0",
+  "version": "4.6.2",
   "description": "",
   "main": "./src/index.js",
   "scripts": {

package/src/app.js

@@ -19,7 +19,7 @@ class App extends Router {
 
         const timeout = setTimeout(() => {
             controller.abort(ErrorResponse.status(504));
-        }, this.timeout);
+        }, this.requestTimeout);
 
         let handleAbort;
         const timeoutPromise = new Promise((_, reject) => {
@@ -27,7 +27,13 @@ class App extends Router {
             signal.addEventListener("abort", handleAbort, { once: true });
         });
 
-        await Promise.race([handlerPromise(), timeoutPromise]).finally(() => {
+        const promise = handlerPromise().catch((error) => {
+            if (!signal.aborted) {
+                controller.abort(error);
+            }
+        });
+
+        await Promise.race([promise, timeoutPromise]).finally(() => {
             signal.removeEventListener("abort", handleAbort);
             clearTimeout(timeout);
         });
@@ -48,7 +54,7 @@ class App extends Router {
                     await handler(req, res);
                 }
             };
-            if (this.timeout) await this.withTimeout(req, handlerPromise);
+            if (this.requestTimeout) await this.withTimeout(req, handlerPromise);
             else await handlerPromise();
         } catch (error) {
             err = error;

package/src/request.js

@@ -21,7 +21,7 @@ class Request extends http.IncomingMessage {
     get query() {
         if (this._query) return this._query;
         if (!this._parsed) {
-            this._query = {};
+            this._query = Object.create(null);
             return this._query;
         }
         this._query = parseQueryString(this._parsed.queryString);

package/src/router.js

@@ -1,10 +1,11 @@
 const { Node } = require("./node.js");
 const { Cache } = require("./cache.js");
+const {  decodePathParam } = require("./util.js");
 
 class Router {
     constructor(options = {}) {
-        this.timeout = options.timeout ?? 0;
-        this.cacheSize = options.size ?? 1000;
+        this.requestTimeout = options.requestTimeout ?? 0;
+        this.cacheSize = options.cacheSize ?? 1000;
         this.root = new Node();
         this.seq = 0;
         this.cache = new Cache(this.cacheSize);
@@ -103,12 +104,7 @@ class Router {
             if (match) {
                 matched = child;
                 for (const name in match.groups) {
-                    const raw = match.groups[name];
-                    try {
-                        params[name] = decodeURIComponent(raw);
-                    } catch {
-                        params[name] = raw;
-                    }
+                    params[name] = decodePathParam(match.groups[name]);
                 }
                 break;
             }

package/src/util.js

@@ -1,23 +1,31 @@
 const URL_REGEX = /^([^?#]*)(\?([^#]*))?(#.*)?$/;
 const QUERY_STRING_REGEX = /([^=&]*)?=([^&]*)?/g;
 
+function decodeQueryParam(encoded) {
+    try {
+        return decodeURIComponent(encoded.replace(/\+/g, " "));
+    } catch (error) {
+        return encoded;
+    }
+}
+exports.decodeQueryParam = decodeQueryParam;
+
+function decodePathParam(encoded) {
+    try {
+        return decodeURIComponent(encoded);
+    } catch {
+        return encoded;
+    }
+}
+exports.decodePathParam=decodePathParam
+
 function parseQueryString(queryString) {
     const result = Object.create(null);
     let match;
     while ((match = QUERY_STRING_REGEX.exec(queryString)) !== null) {
         const [, rawName = "", rawValue = ""] = match;
-        let name;
-        try {
-            name = decodeURIComponent(rawName);
-        } catch (error) {
-            name = rawName;
-        }
-        let value;
-        try {
-            value = decodeURIComponent(rawValue);
-        } catch (error) {
-            value = rawValue;
-        }
+        const name = decodeQueryParam(rawName);
+        const value = decodeQueryParam(rawValue);
         if (Object.hasOwn(result, name)) {
             if (Array.isArray(result[name])) result[name].push(value);
             else result[name] = [result[name], value];