#fix: correct buffer send; add node 20 to the ci

metrico · Oct 6, 2023 · 228e17d · 228e17d
1 parent b2e906e
commit 228e17d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/.github/workflows/node-clickhouse.js.yml b/.github/workflows/node-clickhouse.js.yml
@@ -23,7 +23,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [18, 16.x]
+        node-version: [18, 16.x, 20]
         # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
 
     steps:

diff --git a/lib/bun_wrapper.js b/lib/bun_wrapper.js
@@ -26,6 +26,7 @@ const wrapper = (handler, parsers) => {
     let status = 200
     let reqBody = ''
     let headers = {}
+    log.info(`${ctx.url}`)
 
     const stream = new BodyStream()
     setTimeout(async () => {
@@ -115,7 +116,7 @@ const wrapper = (handler, parsers) => {
     if (result) {
       response = result
     }
-    if (response instanceof Object && typeof response !== 'string') {
+    if (response instanceof Object && typeof response !== 'string' && !Buffer.isBuffer(response)) {
       response = JSON.stringify(response)
     }
     return new Response(response, { status: status, headers: headers })