@server/next 0.15.2 → 0.16.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@server/next",
-  "version": "0.15.2",
+  "version": "0.16.0",
   "description": "An experimental reimplementation of server.js focused on the DX",
   "homepage": "https://node-server.com/",
   "repository": "https://github.com/franciscop/server-next.git",

package/readme.md

@@ -21,7 +21,7 @@ app([
 ]);
 ```
 
-It includes all the things you would expect from a modern Server framework, like routing, static file serving, body+file parsing, gzip+brotli\*, streaming, etc.
+It includes all the things you would expect from a modern Server framework, like routing, static file serving, body+file parsing, gzip+brotli, streaming, server-timing, plugins\*, etc.
 
 > \* not yet available
 

package/src/RequestLogger.js

@@ -17,13 +17,6 @@ const format = (n, [below, above], limit = Infinity) => {
   }
 };
 
-const findSize = ({ headers, body, size }, res) => {
-  if (size) return size;
-  if (body) return body.length;
-  if (headers["content-length"]) return +headers["content-length"];
-  return 0;
-};
-
 function simpleType(type) {
   const simpler = {
     "text/html": "html",
@@ -59,7 +52,7 @@ export default function RequestLogger(ctx) {
       const statColor =
         status < 300 ? "green" : status < 500 ? "yellow" : "red";
       const statusBlock = `{${statColor}}[${ctx.res.status}]{/}`;
-      const resSize = format(findSize(ctx.res), ["b", "kb"], 100000);
+      const resSize = format(ctx.res.size || 0, ["b", "kb"], 100000);
       const t = Math.round(ctx.time._total - ctx.time._init);
       const resTime = format(t, ["ms", "s"], 1000);
 

package/src/ServerUrl.js

@@ -36,7 +36,11 @@ export default class ServerUrl extends URL {
 
     for (let key of properties) {
       const value = key in custom ? custom[key] : this[key];
-      Object.defineProperty(this, key, { value, enumerable: true });
+      Object.defineProperty(this, key, {
+        value,
+        enumerable: true,
+        writable: true,
+      });
     }
   }
 

package/src/index.js

@@ -2,7 +2,9 @@ import http from "node:http";
 import fs from "node:fs";
 import fsp from "node:fs/promises";
 import path from "node:path";
+import zlib from "node:zlib";
 import { pipeline } from "node:stream/promises";
+import { Readable, PassThrough } from "node:stream";
 
 import "dotenv/config";
 
@@ -29,6 +31,35 @@ const exists = (file) => {
   );
 };
 
+const measure = (ctx) => {
+  const sizeUp = new PassThrough();
+  ctx.res.size = 0;
+  sizeUp.on("data", (chunk) => {
+    ctx.res.size += chunk.length;
+  });
+  return sizeUp;
+};
+
+const findEncoding = (acceptEncoding) => {
+  let encoding;
+  if (/\bbr\b/.test(acceptEncoding)) {
+    encoding = "br";
+  } else if (/\bgzip\b/.test(acceptEncoding)) {
+    encoding = "gzip";
+  } else if (/\bdeflate\b/.test(acceptEncoding)) {
+    encoding = "deflate";
+  }
+
+  const methods = {
+    deflate: () => zlib.createDeflate(),
+    gzip: () => zlib.createGzip(),
+    br: () => zlib.createBrotliCompress(),
+  };
+
+  const compress = methods[encoding];
+  return [encoding, compress];
+};
+
 const api = {
   get: [],
   post: [],
@@ -119,7 +150,6 @@ export default function (options = {}, plugins) {
           ctx.res.body = out;
           const isHtml = out.trim().startsWith("<");
           ctx.res.headers["content-type"] = isHtml ? "text/html" : "text/plain";
-          ctx.res.headers["content-length"] = Buffer.byteLength(out);
         } else {
           if (out.pipe) {
             ctx.res.body = out;
@@ -127,7 +157,6 @@ export default function (options = {}, plugins) {
           } else {
             // Plain object
             if (out.type) ctx.res.headers["content-type"] = out.type;
-            if (out.length) ctx.res.headers["content-length"] = out.length;
             ctx.res.headers = { ...ctx.res.headers, ...(out.headers || {}) };
             ctx.res.body = out.body || "";
             ctx.res.status = out.status || 200;
@@ -147,21 +176,30 @@ export default function (options = {}, plugins) {
         ctx.res.headers["server-timing"] += ", ";
       }
     });
+
+    const [encoding, compress] = findEncoding(ctx.headers["accept-encoding"]);
+    if (encoding) {
+      ctx.res.headers["content-encoding"] = encoding;
+    }
+
     res.writeHead(ctx.res.status, ctx.res.headers);
-    if (ctx.res.body.pipe) {
-      if (ctx.res.body.path) {
-        ctx.res.type = ctx.res.body.path.split(".").pop();
-        ctx.res.size = await fsp.stat(ctx.res.body.path).then((s) => s.size);
-      } else {
-        ctx.res.size = 0;
-        ctx.res.body.on("data", function (chunk) {
-          ctx.res.size += chunk.length;
-        });
-      }
-      await pipeline(ctx.res.body, res);
+
+    // If it's not a pipe, e.g. a String, make it a pipe
+    if (!ctx.res.body.pipe) {
+      ctx.res.body = Readable.from([ctx.res.body]);
+    }
+
+    if (ctx.res.body.path) {
+      ctx.res.type = ctx.res.body.path.split(".").pop();
+    }
+
+    if (compress) {
+      await pipeline(ctx.res.body, compress(), measure(ctx), res);
     } else {
-      res.end(ctx.res.body);
+      await pipeline(ctx.res.body, measure(ctx), res);
     }
+    res.end();
+
     // The actual sent headers, as seen by the response
     ctx.res.headers = Object.fromEntries(
       res._header