ultimate-express 1.4.7 → 1.4.8

package/package.json

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

package/src/response.js

@@ -153,7 +153,7 @@ module.exports = class Response extends Writable {
                 this.#pendingChunks.push(chunk);
                 const size = this.#pendingChunks.reduce((acc, chunk) => acc + chunk.byteLength, 0);
                 const now = Date.now();
-                // the first chunk is set immediately (!this.#lastWriteChunkTime)
+                // the first chunk is sent immediately (!this.#lastWriteChunkTime)
                 // the other chunks are sent when watermark is reached (size >= HIGH_WATERMARK) 
                 // or if elapsed 100ms of last send (now - this.#lastWriteChunkTime > 100)
                 if (!this.#lastWriteChunkTime || size >= HIGH_WATERMARK || now - this.#lastWriteChunkTime > 100) {

package/src/router.js

@@ -365,7 +365,7 @@ module.exports = class Router extends EventEmitter {
         req.route = route;
         if(route.optimizedParams) {
             req.params = {...req.optimizedParams};
-        } else if(typeof route.path === 'string' && (route.path.includes(':') || route.path.includes('*')) && route.pattern instanceof RegExp) {
+        } else if(typeof route.path === 'string' && (route.path.includes(':') || route.path.includes('*') || (route.path.includes('(') && route.path.includes(')'))) && route.pattern instanceof RegExp) {
             let path = req._originalPath;
             if(req._stack.length > 0) {
                 path = path.replace(this.getFullMountpath(req), '');

package/src/utils.js

@@ -57,7 +57,7 @@ function patternToRegex(pattern, isPrefix = false) {
         .replaceAll('*', '(.*)') // Convert * to .*
         .replace(/\/:(\w+)(\(.+?\))?\??/g, (match, param, regex) => {
             const optional = match.endsWith('?');
-            return `\\/${optional ? '?' : ''}?(?<${param}>${regex ? regex + '($|\\/)' : '[^/]+'})${optional ? '?' : ''}`;
+            return `\\/${optional ? '?' : ''}(?<${param}>${regex ? regex + '($|\\/)' : '[^/]+'})${optional ? '?' : ''}`;
         }); // Convert :param to capture group
 
     return new RegExp(`^${regexPattern}${isPrefix ? '(?=$|\/)' : '$'}`);