braid-http 0.3.18 → 0.3.20

package/braid-http-client.js

@@ -230,7 +230,7 @@ async function braid_fetch (url, params = {}) {
     // Now we define the subscription function we just used:
     function start_subscription (cb, error) {
         if (!res.ok)
-            throw new Error('Request returned not ok', res)
+            throw new Error('Request returned not ok status:', res.status)
 
         if (res.bodyUsed)
             // TODO: check if this needs a return

package/braid-http-server.js

@@ -236,10 +236,22 @@ function braidify (req, res, next) {
             res.statusCode = 209
             res.setHeader("subscribe", req.headers.subscribe)
             res.setHeader('cache-control', 'no-cache, no-transform')
-            if (req.httpVersionMajor == 1) {
-                // Explicitly disable transfer-encoding chunked for http 1
-                res.setHeader('transfer-encoding', '')
-            }
+
+
+            // Note: I used to explicitly disable transfer-encoding chunked
+            // here by setting the header to empty string.  This is the only
+            // way I know to disable it in nodejs.  We don't need chunked
+            // encoding in subscriptions, because chunked encoding is used to
+            // signal the end of a response, and subscriptions don't end.  I
+            // disabled them to make responses cleaner.  However, it turns out
+            // the Caddy proxy throws an error if it receives a response with
+            // transfer-encoding: set to the empty string.  So I'm disabling
+            // it now.
+
+            // if (req.httpVersionMajor == 1) {
+            //     // Explicitly disable transfer-encoding chunked for http 1
+            //     res.setHeader('transfer-encoding', '')
+            // }
 
             // Tell nginx not to buffer the subscription
             res.setHeader('X-Accel-Buffering', 'no')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "braid-http",
-  "version": "0.3.18",
+  "version": "0.3.20",
   "description": "An implementation of Braid-HTTP for Node.js and Browsers",
   "scripts": {
     "test": "node test/server.js"