ultimate-express 1.4.9 → 1.4.10

package/README.md

@@ -35,6 +35,7 @@ Tested using [wrk](https://github.com/wg/wrk) (`-d 60 -t 1 -c 200`). Tested on U
 | middlewares/express-static (/static/index.js) | 6.58k           | 32.45k           | 10.15 MB/sec       | 49.43 MB/sec        | **4.87X**        |
 | engines/ejs (/test)                           | 5.50k           | 40.82k           | 2.45 MB/sec        | 18.38 MB/sec        | **7.42X**        |
 | middlewares/body-urlencoded (/abc)            | 8.07k           | 50.52k           | 1.68 MB/sec        | 10.78 MB/sec        | **6.26X**        |
+| middlewares/compression-file (/small-file)    | 4.81k           | 14.92k           | 386 MB/sec         | 1.17 GB/sec         | **3.10X**        |
 
 ### Performance against other frameworks
 
@@ -327,6 +328,8 @@ Almost all middlewares that are compatible with Express are compatible with µEx
 - ✅ [passport](https://www.npmjs.com/package/passport)
 - ✅ [morgan](https://www.npmjs.com/package/morgan)
 - ✅ [swagger-ui-express](https://www.npmjs.com/package/swagger-ui-express)
+- ✅ [graphql-http](https://www.npmjs.com/package/graphql-http)
+- ✅ [better-sse](https://www.npmjs.com/package/better-sse)
 
 Middlewares and modules that are confirmed to not work:
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ultimate-express",
-  "version": "1.4.9",
+  "version": "1.4.10",
   "description": "The Ultimate Express. Fastest http server with full Express compatibility, based on uWebSockets.",
   "main": "src/index.js",
   "scripts": {

package/src/declarative.js

@@ -31,7 +31,7 @@ module.exports = function compileDeclarative(cb, app) {
 
         const tokens = [...acorn.tokenizer(code, { ecmaVersion: "latest" })];
 
-        if(tokens.some(token => ['throw', 'new', 'await'].includes(token.value))) {
+        if(tokens.some(token => ['throw', 'new', 'await', 'return'].includes(token.value))) {
             return false;
         }
 
@@ -186,7 +186,7 @@ module.exports = function compileDeclarative(cb, app) {
                 if(call.arguments[0].type !== 'Literal' || call.arguments[1].type !== 'Literal') {
                     return false;
                 }
-                const sameHeader = headers.find(header => header[0] === call.arguments[0].value);
+                const sameHeader = headers.find(header => header[0].toLowerCase() === call.arguments[0].value.toLowerCase());
                 if(sameHeader) {
                     sameHeader[1] = call.arguments[1].value;
                 } else {
@@ -207,6 +207,12 @@ module.exports = function compileDeclarative(cb, app) {
                 if(sendUsed) {
                     return false;
                 }
+                if(call.obj.propertyName === 'send') {
+                    const index = headers.findIndex(header => header[0].toLowerCase() === 'content-type');
+                    if(index === -1) {
+                        headers.push(['content-type', 'text/html; charset=utf-8']);
+                    }
+                }
                 const arg = call.arguments[0];
                 if(arg) {
                     if(arg.type === 'Literal') {
@@ -216,6 +222,17 @@ module.exports = function compileDeclarative(cb, app) {
                         let val = arg.value;
                         if(val === null) {
                             val = '';
+                            const index = headers.findIndex(header => header[0].toLowerCase() === 'content-type');
+                            if(index !== -1) {
+                                headers.splice(index, 1);
+                            }
+                        }
+                        if(typeof val === 'boolean') {
+                            if(!headers.some(header => header[0].toLowerCase() === 'content-type')) {
+                                headers.push(['content-type', 'application/json; charset=utf-8']);
+                            } else {
+                                headers.find(header => header[0].toLowerCase() === 'content-type')[1] = 'application/json; charset=utf-8';
+                            }
                         }
                         body.push({type: 'text', value: val});
                     } else if(arg.type === 'TemplateLiteral') {
@@ -304,7 +321,11 @@ module.exports = function compileDeclarative(cb, app) {
                             return false;
                         }
 
-                        headers.push(['content-type', 'application/json; charset=utf-8']);
+                        if(!headers.some(header => header[0].toLowerCase() === 'content-type')) {
+                            headers.push(['content-type', 'application/json; charset=utf-8']);
+                        } else {
+                            headers.find(header => header[0].toLowerCase() === 'content-type')[1] = 'application/json; charset=utf-8';
+                        }
                         body.push({
                             type: 'text',
                             value: 
@@ -334,6 +355,9 @@ module.exports = function compileDeclarative(cb, app) {
             if(header[0].toLowerCase() === 'content-length') {
                 return false;
             }
+            if(header[0].toLowerCase() === 'content-type' && header[1].includes('text/') && !header[1].includes(';')) {
+                header[1] += '; charset=utf-8';
+            }
             decRes = decRes.writeHeader(header[0], header[1]);
         }
 

package/src/middlewares.js

@@ -169,7 +169,7 @@ function createBodyParser(defaultType, beforeReturn) {
 
             req.body = new NullObject();
 
-            // skip reading body for non-json content type
+            // skip reading body for no content type
             if(!type) {
                 return next();
             }
@@ -281,6 +281,10 @@ const json = createBodyParser('application/json', function(req, res, next, optio
             return next(new Error('Invalid body'));
         }
     }
+    if(buf.length === 0) {
+        req.body = {};
+        return next();
+    }
     try {
         req.body = JSON.parse(buf.toString(), options.reviver);
     } catch(e) {

package/src/response.js

@@ -177,6 +177,7 @@ module.exports = class Response extends Writable {
                             }
                         });
                     }, 50);
+                    this.#writeTimeout.unref();
                 }
                 this.writingChunk = false;
                 callback(null);
@@ -243,9 +244,6 @@ module.exports = class Response extends Writable {
                 this._res.writeHeader(header, value);
             }
         }
-        if(!this.headers['content-type']) {
-            this._res.writeHeader('content-type', 'text/html' + (utf8 ? `; charset=utf-8` : ''));
-        }
         this.headersSent = true;
     }
     _implicitHeader() {
@@ -285,6 +283,8 @@ module.exports = class Response extends Writable {
                     this._res.end();
                     this.finished = true;
                     if(this.socketExists) this.socket.emit('close');
+                    this.emit('finish');
+                    this.emit('close');
                     return;
                 }
             }
@@ -323,6 +323,7 @@ module.exports = class Response extends Writable {
         const isBuffer = Buffer.isBuffer(body);
         if(body === null || body === undefined) {
             body = '';
+            return this.end(body);
         } else if(typeof body === 'object' && !isBuffer) {
             return this.json(body);
         } else if(typeof body === 'number') {
@@ -331,18 +332,25 @@ module.exports = class Response extends Writable {
                 return this.status(body).send(arguments[1]);
             } else {
                 deprecated('res.send(status)', 'res.sendStatus(status)');
+                if(!this.headers['content-type']) {
+                    this.headers['content-type'] = 'text/plain; charset=utf-8';
+                }
                 return this.sendStatus(body);
             }
+        } else if(typeof body === 'boolean') {
+            return this.json(body);
         } else if(!isBuffer) {
             body = String(body);
         }
-        if(typeof body === 'string') {
+        if(typeof body === 'string' && !isBuffer) {
             const contentType = this.headers['content-type'];
             if(!contentType) {
                 this.headers['content-type'] = 'text/html; charset=utf-8';
             } else if(!contentType.includes(';')) {
                 this.headers['content-type'] += '; charset=utf-8';
             }
+        } else {
+            this.headers['content-type'] = 'application/octet-stream';
         }
         return this.end(body);
     }
@@ -611,6 +619,9 @@ module.exports = class Response extends Writable {
     getHeader(field) {
         return this.get(field);
     }
+    getHeaders(){
+        return this.headers;
+    }
     removeHeader(field) {
         delete this.headers[field.toLowerCase()];
         return this;
@@ -714,11 +725,7 @@ module.exports = class Response extends Writable {
     jsonp(object) {
         let callback = this.req.query[this.app.get('jsonp callback name')];
         let body = stringify(object, this.app.get('json replacer'), this.app.get('json spaces'), this.app.get('json escape'));
-
-        if(!this.headers['content-type']) {
-            this.headers['content-type'] = 'application/javascript; charset=utf-8';
-            this.headers['X-Content-Type-Options'] = 'nosniff';
-        }
+        let js = false;
 
         if(Array.isArray(callback)) {
             callback = callback[0];
@@ -736,6 +743,13 @@ module.exports = class Response extends Writable {
                     .replace(/\u2029/g, '\\u2029')
             }
             body = '/**/ typeof ' + callback + ' === \'function\' && ' + callback + '(' + body + ');';
+            js = true;
+        }
+
+        
+        if(!this.headers['content-type']) {
+            this.headers['content-type'] = `${js ? 'text/javascript' : 'application/json'}; charset=utf-8`;
+            if(js) this.headers['X-Content-Type-Options'] = 'nosniff';
         }
 
         return this.send(body);
@@ -760,7 +774,25 @@ module.exports = class Response extends Writable {
         this.location(url);
         this.status(status);
         this.headers['content-type'] = 'text/plain; charset=utf-8';
-        return this.send(`${statuses.message[status] ?? status}. Redirecting to ${url}`);
+        let body;
+        // Support text/{plain,html} by default
+        this.format({
+            text: function() {
+                body = statuses.message[status] + '. Redirecting to ' + url
+            },
+            html: function() {
+                let u = escapeHtml(url);
+                body = '<p>' + statuses.message[status] + '. Redirecting to ' + u + '</p>'
+            },
+            default: function() {
+                body = '';
+            }
+        });
+        if (this.req.method === 'HEAD') {
+            this.end();
+        } else {
+            this.end(body);
+        }
     }
 
     type(type) {
@@ -773,6 +805,11 @@ module.exports = class Response extends Writable {
     contentType = this.type;
 
     vary(field) {
+        // checks for back-compat
+        if (!field || (Array.isArray(field) && !field.length)) {
+            deprecate('res.vary(): Provide a field name');
+            return this;
+        }
         vary(this, field);
         return this;
     }

package/src/router.js

@@ -180,7 +180,7 @@ module.exports = class Router extends EventEmitter {
                         optimizedPathToRouter = optimizedPathToRouter.slice(0, -1); // remove last element, which is the router itself
                         if(optimizedPathToRouter) {
                             // wait for routes in router to be registered
-                            setTimeout(() => {
+                            const t = setTimeout(() => {
                                 if(!this.listenCalled) {
                                     return; // can only optimize router whos parent is listening
                                 }
@@ -208,6 +208,7 @@ module.exports = class Router extends EventEmitter {
                                     }
                                 }
                             }, 100);
+                            t.unref();
                         }
                         // only 1 router can be optimized per route
                         break;
@@ -354,7 +355,10 @@ module.exports = class Router extends EventEmitter {
             path = path.slice(0, -1);
         }
         let match = pattern.exec(path);
-        const obj = match?.groups ?? new NullObject();
+        if( match?.groups ){
+            return match.groups;
+        }
+        const obj = new NullObject();
         for(let i = 1; i < match.length; i++) {
             obj[i - 1] = match[i];
         }