@middy/http-content-encoding 5.0.3 → 5.2.0

package/index.js

@@ -1,63 +1,103 @@
-import { Readable } from 'node:stream';
-import { createBrotliCompress as brotliCompressStream, createGzip as gzipCompressStream, createDeflate as deflateCompressStream } from 'node:zlib';
-import { normalizeHttpResponse } from '@middy/util';
+import { Readable } from 'node:stream'
+
+import {
+  createBrotliCompress as brotliCompressStream,
+  createGzip as gzipCompressStream,
+  createDeflate as deflateCompressStream
+} from 'node:zlib'
+
+import { normalizeHttpResponse } from '@middy/util'
+
 const contentEncodingStreams = {
-    br: brotliCompressStream,
-    gzip: gzipCompressStream,
-    deflate: deflateCompressStream
-};
+  br: brotliCompressStream,
+  gzip: gzipCompressStream,
+  deflate: deflateCompressStream
+}
+
 const defaults = {
-    br: undefined,
-    gzip: undefined,
-    deflate: undefined,
-    overridePreferredEncoding: []
-};
-const httpContentEncodingMiddleware = (opts)=>{
-    const options = {
-        ...defaults,
-        ...opts
-    };
-    const supportedContentEncodings = Object.keys(contentEncodingStreams);
-    const httpContentEncodingMiddlewareAfter = async (request)=>{
-        normalizeHttpResponse(request);
-        const { context: { preferredEncoding, preferredEncodings }, response } = request;
-        if (response.isBase64Encoded || !preferredEncoding || !supportedContentEncodings.includes(preferredEncoding) || !response.body) {
-            return;
-        }
-        let contentEncodingStream = contentEncodingStreams[preferredEncoding](options[preferredEncoding]);
-        let contentEncoding = preferredEncoding;
-        for (const encoding of options.overridePreferredEncoding){
-            if (!preferredEncodings.includes(encoding)) continue;
-            contentEncodingStream = contentEncodingStreams[encoding](options[encoding]);
-            contentEncoding = encoding;
-            break;
-        }
-        if (response.body?._readableState) {
-            request.response.headers['Content-Encoding'] = contentEncoding;
-            request.response.body.pipe(contentEncodingStream);
-            return;
-        }
-        const stream = Readable.from(response.body).pipe(contentEncodingStream);
-        const chunks = [];
-        for await (const chunk of stream){
-            chunks.push(chunk);
-        }
-        const body = Buffer.concat(chunks).toString('base64');
-        if (body.length < response.body.length) {
-            response.headers['Content-Encoding'] = contentEncoding;
-            response.body = body;
-            response.isBase64Encoded = true;
-        }
-        request.response = response;
-    };
-    const httpContentEncodingMiddlewareOnError = async (request)=>{
-        if (typeof request.response === 'undefined') return;
-        httpContentEncodingMiddlewareAfter(request);
-    };
-    return {
-        after: httpContentEncodingMiddlewareAfter,
-        onError: httpContentEncodingMiddlewareOnError
-    };
-};
-export default httpContentEncodingMiddleware;
+  br: undefined,
+  // zstd: undefined,
+  gzip: undefined,
+  deflate: undefined,
+  overridePreferredEncoding: []
+}
+
+/*
+ * `zstd` disabled due to lack of support in browsers
+ * https://github.com/Fyrd/caniuse/issues/4065
+ * https://github.com/andrew-aladev/brotli-vs-zstd
+ */
+
+const httpContentEncodingMiddleware = (opts) => {
+  const options = { ...defaults, ...opts }
+
+  const supportedContentEncodings = Object.keys(contentEncodingStreams)
+
+  const httpContentEncodingMiddlewareAfter = async (request) => {
+    normalizeHttpResponse(request)
+    const {
+      context: { preferredEncoding, preferredEncodings },
+      response
+    } = request
+
+    // Encoding not supported OR already encoded
+    if (
+      response.isBase64Encoded ||
+      !preferredEncoding ||
+      !supportedContentEncodings.includes(preferredEncoding) ||
+      !response.body
+    ) {
+      return
+    }
+
+    let contentEncodingStream = contentEncodingStreams[preferredEncoding](
+      options[preferredEncoding]
+    )
+    let contentEncoding = preferredEncoding
+    for (const encoding of options.overridePreferredEncoding) {
+      if (!preferredEncodings.includes(encoding)) continue
+      contentEncodingStream = contentEncodingStreams[encoding](
+        options[encoding]
+      )
+      contentEncoding = encoding
+      break
+    }
+
+    // Support streamifyResponse
+    if (response.body?._readableState) {
+      request.response.headers['Content-Encoding'] = contentEncoding
+      request.response.body.pipe(contentEncodingStream)
+      return
+    }
+
+    const stream = Readable.from(response.body).pipe(contentEncodingStream)
+
+    const chunks = []
+    for await (const chunk of stream) {
+      chunks.push(chunk)
+    }
+    // TODO update to btoa if faster
+    const body = Buffer.concat(chunks).toString('base64')
+
+    // Only apply encoding if it's smaller
+    if (body.length < response.body.length) {
+      response.headers['Content-Encoding'] = contentEncoding
+      response.body = body
+      response.isBase64Encoded = true
+    }
+
+    request.response = response
+  }
+
+  const httpContentEncodingMiddlewareOnError = async (request) => {
+    if (typeof request.response === 'undefined') return
+    httpContentEncodingMiddlewareAfter(request)
+  }
+
+  return {
+    after: httpContentEncodingMiddlewareAfter,
+    onError: httpContentEncodingMiddlewareOnError
+  }
+}
 
+export default httpContentEncodingMiddleware

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@middy/http-content-encoding",
-  "version": "5.0.3",
+  "version": "5.2.0",
   "description": "Http content encoding middleware for the middy framework",
   "type": "module",
   "engines": {
@@ -63,10 +63,10 @@
     "url": "https://github.com/sponsors/willfarrell"
   },
   "dependencies": {
-    "@middy/util": "5.0.3"
+    "@middy/util": "5.2.0"
   },
   "devDependencies": {
-    "@middy/core": "5.0.3"
+    "@middy/core": "5.2.0"
   },
-  "gitHead": "87660575a7ac2b52e4153c407a4c63c9449dcd0d"
+  "gitHead": "2d9096a49cd8fb62359517be96d6c93609df41f0"
 }