ultimate-express 1.4.4 → 1.4.5

package/README.md

@@ -308,7 +308,7 @@ Almost all middlewares that are compatible with Express are compatible with µEx
 - ✅ [body-parser](https://npmjs.com/package/body-parser) (use `express.text()` etc instead for better performance)
 - ✅ [cookie-parser](https://npmjs.com/package/cookie-parser)
 - ✅ [cookie-session](https://npmjs.com/package/cookie-session)
-- ✅ [compression](https://npmjs.com/package/compression)
+- 🚧 [compression](https://npmjs.com/package/compression) - in some cases may send uncompressed files
 - ✅ [serve-static](https://npmjs.com/package/serve-static) (use `express.static()` instead for better performance)
 - ✅ [serve-index](https://npmjs.com/package/serve-index)
 - ✅ [cors](https://npmjs.com/package/cors)
@@ -322,6 +322,7 @@ Almost all middlewares that are compatible with Express are compatible with µEx
 - ✅ [express-subdomain](https://npmjs.com/package/express-subdomain)
 - ✅ [vhost](https://npmjs.com/package/vhost)
 - ✅ [tsoa](https://github.com/lukeautry/tsoa)
+- ✅ [express-mongo-sanitize](https://www.npmjs.com/package/express-mongo-sanitize)
 
 Middlewares and modules that are confirmed to not work:
 

package/package.json

@@ -1,11 +1,14 @@
 {
   "name": "ultimate-express",
-  "version": "1.4.4",
+  "version": "1.4.5",
   "description": "The Ultimate Express. Fastest http server with full Express compatibility, based on uWebSockets.",
   "main": "src/index.js",
   "scripts": {
     "test": "node tests/index.js",
-    "dev": "node --inspect=9229 demo/index.js"
+    "dev": "node --inspect=9229 demo/index.js",
+    "cover": "npm run cover:unit && npm run cover:report",
+    "cover:unit": "nyc --silent npm run test",
+    "cover:report": "nyc report --reporter=html"
   },
   "engines": {
     "node": ">=18"
@@ -42,7 +45,7 @@
   "dependencies": {
     "@types/express": "^4.0.0",
     "accepts": "^1.3.8",
-    "acorn": "^8.14.0",
+    "acorn": "^8.14.1",
     "bytes": "^3.1.2",
     "cookie": "^1.0.2",
     "cookie-signature": "^1.2.2",
@@ -54,7 +57,7 @@
     "mime-types": "^2.1.35",
     "ms": "^2.1.3",
     "proxy-addr": "^2.0.7",
-    "qs": "^6.13.1",
+    "qs": "^6.14.0",
     "range-parser": "^1.2.1",
     "statuses": "^2.0.1",
     "tseep": "^1.3.1",
@@ -65,27 +68,28 @@
   "devDependencies": {
     "@codechecks/client": "^0.1.12",
     "body-parser": "^1.20.3",
-    "compression": "^1.7.5",
+    "compression": "^1.8.0",
     "cookie-parser": "^1.4.6",
     "cookie-session": "^2.1.0",
     "cors": "^2.8.5",
     "ejs": "^3.1.10",
     "errorhandler": "^1.5.1",
     "exit-hook": "^2.2.1",
-    "express-4": "npm:express@4",
-    "express-5": "npm:express@5",
+    "express": "latest-4",
     "express-art-template": "^1.0.1",
     "express-async-errors": "^3.1.1",
     "express-dot-engine": "^1.0.8",
     "express-fileupload": "^1.5.1",
-    "express-handlebars": "^8.0.1",
-    "express-rate-limit": "^7.4.1",
+    "express-handlebars": "^8.0.2",
+    "express-mongo-sanitize": "^2.2.0",
+    "express-rate-limit": "^7.5.0",
     "express-session": "^1.18.0",
     "express-subdomain": "^1.0.6",
     "formdata-node": "^6.0.3",
     "method-override": "^3.0.0",
     "multer": "^1.4.5-lts.1",
     "mustache-express": "^1.3.2",
+    "nyc": "^17.1.0",
     "pako": "^2.1.0",
     "pkg-pr-new": "^0.0.29",
     "pug": "^3.0.3",

package/src/request.js

@@ -201,6 +201,9 @@ module.exports = class Request extends Readable {
         return index !== -1 ? header.slice(0, index).trim() : header.trim();
     }
 
+    set query(query) {
+        return this.#cachedQuery = query;
+    }
     get query() {
         if(this.#cachedQuery) {
             return this.#cachedQuery;
@@ -251,7 +254,7 @@ module.exports = class Request extends Readable {
         let ip = '';
         if(this.rawIp.byteLength === 4) {
             // ipv4
-            ip = this.rawIp.join('.');
+            ip = new Uint8Array(this.rawIp).join('.');
         } else if(this.rawIp.byteLength === 16) {
             // ipv6
             const dv = new DataView(this.rawIp);
@@ -262,7 +265,7 @@ module.exports = class Request extends Readable {
                 }
             }
         } else {
-            this.#cachedParsedIp = undefined; // unix sockets dont have ip
+            ip = undefined; // unix sockets dont have ip
         }
         this.#cachedParsedIp = ip;
         return ip;
@@ -348,6 +351,9 @@ module.exports = class Request extends Readable {
         return parseRange(size, range, options);
     }
 
+    set headers(headers) {
+        this.#cachedHeaders = headers;
+    }
     get headers() {
         // https://nodejs.org/api/http.html#messageheaders
         if(this.#cachedHeaders) {

package/src/response.js

@@ -81,6 +81,7 @@ module.exports = class Response extends Writable {
         this.statusText = undefined;
         this.chunkedTransfer = true;
         this.totalSize = 0;
+        this.writingChunk = false;
         this.headers = {
             'connection': 'keep-alive',
             'keep-alive': 'timeout=10'
@@ -121,62 +122,62 @@ module.exports = class Response extends Writable {
     }
 
     _write(chunk, encoding, callback) {
-        if(this.aborted) {
+        if (this.aborted) {
             const err = new Error('Request aborted');
             err.code = 'ECONNABORTED';
             return this.destroy(err);
         }
-        if(this.finished) {
+        if (this.finished) {
             const err = new Error('Response already finished');
             return this.destroy(err);
         }
+
+        this.writingChunk = true;
         this._res.cork(() => {
-            if(!this.headersSent) {
+            if (!this.headersSent) {
                 this.writeHead(this.statusCode);
                 const statusMessage = this.statusText ?? statuses.message[this.statusCode] ?? '';
                 this._res.writeStatus(`${this.statusCode} ${statusMessage}`.trim());
                 this.writeHeaders(typeof chunk === 'string');
             }
-            if(!Buffer.isBuffer(chunk) && !(chunk instanceof ArrayBuffer)) {
+    
+            if (!Buffer.isBuffer(chunk) && !(chunk instanceof ArrayBuffer)) {
                 chunk = Buffer.from(chunk);
                 chunk = chunk.buffer.slice(chunk.byteOffset, chunk.byteOffset + chunk.byteLength);
             }
-            if(this.chunkedTransfer) {
-                // chunked transfer encoding
+    
+            if (this.chunkedTransfer) {
                 this._res.write(chunk);
-                callback();
+                this.writingChunk = false;
+                callback(null);
             } else {
-                // fixed size transfer encoding
                 const lastOffset = this._res.getWriteOffset();
                 const [ok, done] = this._res.tryEnd(chunk, this.totalSize);
-                if(done) {
-                    this.destroy();
+                if (done) {
+                    super.end();
                     this.finished = true;
-                    if(this.socketExists) this.socket.emit('close');
-                    callback();
+                    this.writingChunk = false;
+                    if (this.socketExists) this.socket.emit('close');
+                    callback(null);
+                } else if (!ok) {
+                    this._res.ab = chunk;
+                    this._res.abOffset = lastOffset;
+                    this._res.onWritable((offset) => {
+                        if (this.finished) return true;
+                        const [ok, done] = this._res.tryEnd(this._res.ab.slice(offset - this._res.abOffset), this.totalSize);
+                        if (done) {
+                            this.finished = true;
+                            if (this.socketExists) this.socket.emit('close');
+                        }
+                        if (ok) {
+                            this.writingChunk = false;
+                            callback(null);
+                        }
+                        return ok;
+                    });
                 } else {
-                    // still writing
-                    if(!ok) {
-                        // wait until uWS is ready to accept more data
-                        this._res.onWritable((offset) => {
-                            if(this.finished) {
-                                return true;
-                            }
-                            const [ok, done] = this._res.tryEnd(chunk.slice(offset - lastOffset), this.totalSize);
-                            if(done) {
-                                this.destroy();
-                                this.finished = true;
-                                if(this.socketExists) this.socket.emit('close');
-                                callback();
-                            } else if(ok) {
-                                callback();
-                            }
-                            
-                            return ok;
-                        });
-                    } else {
-                        callback();
-                    }
+                    this.writingChunk = false;
+                    callback(null);
                 }
             }
         });
@@ -229,6 +230,12 @@ module.exports = class Response extends Writable {
         return this.status(code).send(statuses.message[+code] ?? code.toString());
     }
     end(data) {
+        if(this.writingChunk) {
+            this.once('drain', () => {
+                this.end(data);
+            });
+            return;
+        }
         if(this.finished) {
             return;
         }
@@ -498,7 +505,8 @@ module.exports = class Response extends Writable {
                 opts.end = Math.max(offset, offset + len - 1);
             }
             const file = fs.createReadStream(fullpath, opts);
-            pipeStreamOverResponse(this, file, len, callback);
+            this.set('Content-Length', len);
+            file.pipe(this);
         }
     }
     download(path, filename, options, callback) {
@@ -739,60 +747,4 @@ module.exports = class Response extends Writable {
     get writableFinished() {
         return this.finished;
     }
-}
-
-function pipeStreamOverResponse(res, readStream, totalSize, callback) {
-    readStream.on('data', (chunk) => {
-        if(res.finished) {
-            return readStream.destroy();
-        }
-        res._res.cork(() => {
-            if(!res.headersSent) {
-                res.writeHead(res.statusCode);
-                const statusMessage = res.statusText ?? statuses.message[res.statusCode] ?? '';
-                res._res.writeStatus(`${res.statusCode} ${statusMessage}`.trim());
-                res.writeHeaders(true);
-            }
-            const ab = chunk.buffer.slice(chunk.byteOffset, chunk.byteOffset + chunk.byteLength);
-        
-            const lastOffset = res._res.getWriteOffset();
-            const [ok, done] = res._res.tryEnd(ab, totalSize);
-      
-            if (done) {
-                readStream.destroy();
-                res.finished = true;
-                if(res.socketExists) res.socket.emit('close');
-                if(callback) callback();
-            } else if (!ok) {
-                readStream.pause();
-        
-                res._res.ab = ab;
-                res._res.abOffset = lastOffset;
-        
-                res._res.onWritable((offset) => {
-                    if(res.finished) {
-                        return true;
-                    }
-                    const [ok, done] = res._res.tryEnd(res._res.ab.slice(offset - res._res.abOffset), totalSize);
-                    if (done) {
-                        readStream.destroy();
-                        res.finished = true;
-                        if(res.socketExists) res.socket.emit('close');
-                        if(callback) callback();
-                    } else if (ok) {
-                        readStream.resume();
-                    }
-            
-                    return ok;
-                });
-            }
-        });
-    }).on('error', e => {
-        if(callback) callback(e);
-        if(!res.finished) {
-            res._res.close();
-            res.finished = true;
-            if(res.socketExists) res.socket.emit('error', e);
-        }
-    });
-}
+}