braid-http 0.1.8 → 0.1.10

package/braid-http-client.js

@@ -357,7 +357,7 @@ var subscription_parser = (cb) => ({
         this.state.input += input
 
         // Now loop through the input and parse until we hit a dead end
-        do {
+        while (this.state.input.trim() !== '') {
             this.state = parse_version (this.state)
 
             // Maybe we parsed a version!  That's cool!
@@ -380,7 +380,8 @@ var subscription_parser = (cb) => ({
             }
 
             // We stop once we've run out of parseable input.
-        } while (this.state.result !== 'waiting' && this.state.input.trim() !== '')
+            if (this.state.result == 'waiting') break
+        }
     }
 })
 
@@ -552,14 +553,14 @@ function parse_body (state) {
 
     // Parse Patches
 
-    else if (state.headers.patches) {
+    else if (state.headers.patches != null) {
         state.patches = state.patches || []
 
         var last_patch = state.patches[state.patches.length-1]
 
         // Parse patches until the final patch has its content filled
         while (!(state.patches.length === state.headers.patches
-                 && 'content' in last_patch)) {
+                 && (state.patches.length === 0 || 'content' in last_patch))) {
 
             state.input = state.input.trimStart()
 

package/braid-http-server.js

@@ -43,9 +43,8 @@ function generate_patches(res, patches) {
     // Build up the string as a result
     var result = ''
 
-    // Add `Patches: N` header if we have multiple patches
-    if (patches.length > 1)
-        result += `Patches: ${patches.length}\r\n\r\n`
+    // Add `Patches: N` header
+    result += `Patches: ${patches.length}\r\n\r\n`
 
     // Generate each patch
     patches.forEach((patch, i) => {
@@ -66,8 +65,17 @@ ${patch.content}`
 function parse_patches (req, cb) {
     var num_patches = req.headers.patches
 
-    // Parse a single patch from the request body
-    if (num_patches === undefined) {
+    // Parse a request body as an "everything" patch
+    if (!num_patches && !req.headers['content-range']) {
+        var body = ''
+        req.on('data', chunk => {body += chunk.toString()})
+        req.on('end', () => {
+            cb([{unit: 'everything', range: '', content: body}])
+        })
+    }
+
+    // Parse a single patch, lacking Patches: N
+    else if (num_patches === undefined && req.headers['content-range']) {
 
         // We only support range patches right now, so there must be a
         // Content-Range header.
@@ -231,6 +239,9 @@ function braidify (req, res, next) {
             res.setHeader('cache-control', 'no-cache, no-transform')
             res.setHeader('transfer-encoding', '')
 
+            // Tell nginx not to buffer the subscription
+            res.setHeader('X-Accel-Buffering', 'no')
+
             var connected = true
             function disconnected (x) {
                 if (!connected) return

package/package.json

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