ultimate-express 1.2.11 → 1.2.12

package/package.json

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

package/src/request.js

@@ -34,6 +34,7 @@ module.exports = class Request extends Readable {
     #cachedHeaders = null;
     #cachedDistinctHeaders = null;
     #rawHeadersEntries = [];
+    #cachedParsedIp = null;
     constructor(req, res, app) {
         super();
         this._res = res;
@@ -58,13 +59,13 @@ module.exports = class Request extends Readable {
         }
         this.method = req.getMethod().toUpperCase();
         this.params = {};
-        this.rawIp = Buffer.from(this._res.getRemoteAddressAsText()).toString();
 
         this._gotParams = new Set();
         this._stack = [];
         this._paramStack = [];
         this.bufferedData = Buffer.allocUnsafe(0);
         this.receivedData = false;
+        this.rawIp = this._res.getRemoteAddress();
 
         const additionalMethods = this.app.get('body methods');
         // skip reading body for non-POST requests
@@ -156,7 +157,7 @@ module.exports = class Request extends Readable {
     get ip() {
         const trust = this.app.get('trust proxy fn');
         if(!trust) {
-            return this.rawIp;
+            return this.parsedIp;
         }
         return proxyaddr(this, trust);
     }
@@ -217,9 +218,31 @@ module.exports = class Request extends Readable {
         return this.headers['x-requested-with'] === 'XMLHttpRequest';
     }
 
+    get parsedIp() {
+        if(this.#cachedParsedIp) {
+            return this.#cachedParsedIp;
+        }
+        let ip = '';
+        if(this.rawIp.byteLength === 4) {
+            // ipv4
+            ip = this.rawIp.join('.');
+        } else {
+            // ipv6
+            const dv = new DataView(this.rawIp);
+            for(let i = 0; i < 8; i++) {
+                ip += dv.getUint16(i * 2).toString(16).padStart(4, '0');
+                if(i < 7) {
+                    ip += ':';
+                }
+            }
+        }
+        this.#cachedParsedIp = ip;
+        return ip;
+    }
+
     get connection() {
         return {
-            remoteAddress: this.rawIp,
+            remoteAddress: this.parsedIp,
             localPort: this.app.port,
             remotePort: this.app.port,
             encrypted: this.app.ssl,

package/src/response.js

@@ -28,7 +28,9 @@ const fs = require("fs");
 const Path = require("path");
 const statuses = require("statuses");
 const { sign } = require("cookie-signature");
-const { EventEmitter } = require("tseep");
+// events is faster at init, tseep is faster at sending events
+// since we create a ton of objects and dont send a ton of events, its better to use events here
+const { EventEmitter } = require("events");
 const http = require("http");
 const ms = require('ms');   
 const etag = require("etag");
@@ -67,8 +69,12 @@ module.exports = class Response extends Writable {
         this.chunkedTransfer = true;
         this.totalSize = 0;
         this.headers = {
+            'connection': 'keep-alive',
             'keep-alive': 'timeout=10'
         };
+        if(this.app.get('x-powered-by')) {
+            this.headers['x-powered-by'] = 'UltimateExpress';
+        }
         // support for node internal
         this[kOutHeaders] = new Proxy(this.headers, {
             set: (obj, prop, value) => {
@@ -80,9 +86,6 @@ module.exports = class Response extends Writable {
             }
         });
         this.body = undefined;
-        if(this.app.get('x-powered-by')) {
-            this.set('x-powered-by', 'uExpress');
-        }
         this.on('error', (err) => {
             if(this.aborted) {
                 return;
@@ -194,7 +197,7 @@ module.exports = class Response extends Writable {
             if(!this.headersSent) {
                 const etagFn = this.app.get('etag fn');
                 if(data && !this.headers['etag'] && etagFn && !this.req.noEtag) {
-                    this.set('etag', etagFn(data));
+                    this.headers['etag'] = etagFn(data);
                 }
                 const fresh = this.req.fresh;
                 this._res.writeStatus(fresh ? '304' : this.statusCode.toString());
@@ -649,7 +652,7 @@ module.exports = class Response extends Writable {
         return this.send(body);
     }
     links(links) {
-        this.set('Link', Object.entries(links).map(([rel, url]) => `<${url}>; rel="${rel}"`).join(', '));
+        this.headers['link'] = Object.entries(links).map(([rel, url]) => `<${url}>; rel="${rel}"`).join(', ');
         return this;
     }
     location(path) {

package/src/router.js

@@ -326,7 +326,6 @@ module.exports = class Router extends EventEmitter {
     }
 
     #preprocessRequest(req, res, route) {
-        return new Promise(async resolve => {
             req.route = route;
             if(route.optimizedParams) {
                 req.params = req.optimizedParams;
@@ -350,34 +349,37 @@ module.exports = class Router extends EventEmitter {
                 }
             }
 
-            for(let param in req.params) {
-                if(this.#paramCallbacks.has(param) && !req._gotParams.has(param)) {
-                    req._gotParams.add(param);
-                    const pcs = this.#paramCallbacks.get(param);
-                    for(let i = 0; i < pcs.length; i++) {
-                        const fn = pcs[i];
-                        await new Promise(resolveRoute => {
-                            const next = (thingamabob) => {
-                                if(thingamabob) {
-                                    if(thingamabob === 'route') {
-                                        return resolve('route');
-                                    } else {
-                                        this.#handleError(thingamabob, req, res);
-                                        return resolve(false);
-                                    }
-                                }
-                                return resolveRoute();
-                            };
-                            req.next = next;
-                            fn(req, res, next, req.params[param], param);
-                        });
+            if(this.#paramCallbacks.size > 0) {
+                return new Promise(async resolve => {
+                    for(let param in req.params) {
+                        if(this.#paramCallbacks.has(param) && !req._gotParams.has(param)) {
+                            req._gotParams.add(param);
+                            const pcs = this.#paramCallbacks.get(param);
+                            for(let i = 0; i < pcs.length; i++) {
+                                const fn = pcs[i];
+                                await new Promise(resolveRoute => {
+                                    const next = (thingamabob) => {
+                                        if(thingamabob) {
+                                            if(thingamabob === 'route') {
+                                                return resolve('route');
+                                            } else {
+                                                this.#handleError(thingamabob, req, res);
+                                                return resolve(false);
+                                            }
+                                        }
+                                        return resolveRoute();
+                                    };
+                                    req.next = next;
+                                    fn(req, res, next, req.params[param], param);
+                                });
+                            }
+                        }
                     }
-                }
+                });
             }
-
-            resolve(true);
-        });
+            return true;
     }
+
     param(name, fn) {
         if(typeof name === 'function') {
             deprecated('app.param(callback)', 'app.param(name, callback)', true);
@@ -413,7 +415,7 @@ module.exports = class Router extends EventEmitter {
                 return resolve(this._routeRequest(req, res, 0, this._routes, false, skipUntil));
             }
             let callbackindex = 0;
-            const continueRoute = await this.#preprocessRequest(req, res, route);
+            let continueRoute = await this.#preprocessRequest(req, res, route);
             if(route.use) {
                 req._stack.push(route.path);
                 req._opPath =