braid-http 1.1.1 → 1.2.0

package/braid-http-client.js

@@ -189,18 +189,32 @@ async function braid_fetch (url, params = {}) {
         if (params.patches.length === 1) {
             let patch = params.patches[0]
             params.headers.set('Content-Range', `${patch.unit} ${patch.range}`)
-            params.headers.set('Content-Length', `${(new TextEncoder().encode(patch.content)).length}`)
+
+            if (typeof patch.content === 'string')
+                patch.content = new TextEncoder().encode(patch.content)
+
+            params.headers.set('Content-Length', `${patch.content.length}`)
             params.body = patch.content
         }
 
         // Multiple patches get sent within a Patches: N block
         else {
             params.headers.set('Patches', params.patches.length)
-            params.body = (params.patches).map(patch => {
-                var length = `content-length: ${(new TextEncoder().encode(patch.content)).length}`
+            let bufs = []
+            let te = new TextEncoder()
+            for (let patch of params.patches) {
+                if (bufs.length) bufs.push(te.encode(`\r\n`))
+
+                if (typeof patch.content === 'string')
+                    patch.content = te.encode(patch.content)
+
+                var length = `content-length: ${patch.content.length}`
                 var range = `content-range: ${patch.unit} ${patch.range}`
-                return `${length}\r\n${range}\r\n\r\n${patch.content}\r\n`
-            }).join('\r\n')
+                bufs.push(te.encode(`${length}\r\n${range}\r\n\r\n`))
+                bufs.push(patch.content)
+                bufs.push(te.encode(`\r\n`))
+            }
+            params.body = new Blob(bufs)
         }
     }
 
@@ -306,10 +320,13 @@ async function braid_fetch (url, params = {}) {
                         // Each time something happens, we'll either get a new
                         // version back, or an error.
                         (result, err) => {
-                            if (!err)
+                            if (!err) {
+                                // check whether we aborted
+                                if (original_signal?.aborted) throw new DOMException('already aborted', 'AbortError')
+
                                 // Yay!  We got a new version!  Tell the callback!
                                 cb(result)
-                            else
+                            } else
                                 // This error handling code runs if the connection
                                 // closes, or if there is unparseable stuff in the
                                 // streamed response.
@@ -486,7 +503,18 @@ var subscription_parser = (cb) => ({
                     }
                 })
 
-                this.cb(update)
+                for (let p of update.patches ?? []) {
+                    Object.defineProperty(p, 'content_text', {
+                        get: () => new TextDecoder('utf-8').decode(p.content)
+                    })
+                }
+
+                try {
+                    this.cb(update)
+                } catch (e) {
+                    this.cb(null, e)
+                    return
+                }
 
                 // Reset the parser for the next version!
                 this.state = {input: this.state.input}
@@ -629,7 +657,7 @@ function parse_body (state) {
             state.patches = [{
                 unit: match.unit,
                 range: match.range,
-                content: (new TextDecoder('utf-8')).decode(new Uint8Array(state.input.slice(0, content_length))),
+                content: new Uint8Array(state.input.slice(0, content_length)),
 
                 // Question: Perhaps we should include headers here, like we do for
                 // the Patches: N headers below?
@@ -715,7 +743,7 @@ function parse_body (state) {
 
                 last_patch.unit = match.unit
                 last_patch.range = match.range
-                last_patch.content = (new TextDecoder('utf-8')).decode(new Uint8Array(state.input.slice(0, content_length)))
+                last_patch.content = new Uint8Array(state.input.slice(0, content_length))
                 last_patch.extra_headers = extra_headers(last_patch.headers)
                 delete last_patch.headers  // We only keep the extra headers ^^
 

package/braid-http-server.js

@@ -66,10 +66,14 @@ ${patch.content}`
 // Deprecated method for legacy support
 function parse_patches (req, cb) {
     parse_update(req, update => {
-        if (typeof update.body === 'string')
+        if (update.body != null) {
             // Return body as an "everything" patch
-            cb([{unit: 'everything', range: '', content: update.body}])
-        else
+            let patch = {unit: 'everything', range: '', content: update.body}
+            Object.defineProperty(patch, 'content_text', {
+                get: () => new TextDecoder('utf-8').decode(patch.content)
+            })
+            cb([patch])
+        } else
             cb(update.patches)
     })
 }
@@ -80,16 +84,20 @@ function parse_update (req, cb) {
     var num_patches = req.headers.patches
 
     if (!num_patches && !req.headers['content-range']) {
-        var body = ''
-        req.on('data', chunk => {body += chunk.toString()})
+        var buffer = []
+        req.on('data', chunk => buffer.push(chunk))
         req.on('end', () => {
-            cb({ body, patches: undefined })
+            let body = new Uint8Array(Buffer.concat(buffer))
+            let update = { body, patches: undefined }
+            Object.defineProperty(update, 'body_text', {
+                get: () => new TextDecoder('utf-8').decode(update.body)
+            })
+            cb(update)
         })
     }
 
     // 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.
         assert(req.headers['content-range'], 'No patches to parse: need `Patches: N` or `Content-Range:` header in ' + JSON.stringify(req.headers))
@@ -104,8 +112,11 @@ function parse_update (req, cb) {
         req.on('data', chunk => buffer.push(chunk))
         // Then return it
         req.on('end', () => {
-            patches = [{unit, range, content: Buffer.concat(buffer).toString('utf8')}]
-            cb({ patches, body: undefined })
+            let patch = {unit, range, content: new Uint8Array(Buffer.concat(buffer))}
+            Object.defineProperty(patch, 'content_text', {
+                get: () => new TextDecoder('utf-8').decode(patch.content)
+            })
+            cb({ patches: [patch], body: undefined })
         })
     }
 
@@ -136,7 +147,7 @@ function parse_update (req, cb) {
                 if (!('content-length' in headers)) {
                     // Print a nice error if it's missing
                     console.error('No content-length in', JSON.stringify(headers),
-                                  'from', {buffer})
+                                  'from', new TextDecoder().decode(new Uint8Array(buffer)), {buffer})
                     process.exit(1)
                 }
 
@@ -150,10 +161,14 @@ function parse_update (req, cb) {
 
                 // Content-range is of the form '<unit> <range>' e.g. 'json .index'
                 var [unit, range] = parse_content_range(headers['content-range'])
-                var patch_content = new TextDecoder('utf-8').decode(new Uint8Array(h.remaining_bytes.slice(0, body_length)))
+                var patch_content = new Uint8Array(h.remaining_bytes.slice(0, body_length))
 
                 // We've got our patch!
-                patches.push({unit, range, content: patch_content})
+                let patch = {unit, range, content: patch_content}
+                Object.defineProperty(patch, 'content_text', {
+                    get: () => new TextDecoder('utf-8').decode(patch.content)
+                })
+                patches.push(patch)
 
                 buffer = h.remaining_bytes.slice(body_length)
             }
@@ -215,7 +230,7 @@ function braidify (req, res, next) {
         (done, err) => parse_patches(
             req,
             (patches) => done(patches.map(
-                p => ({...p, content: JSON.parse(p.content)})
+                p => ({...p, content: JSON.parse(p.content_text)})
             ))
         )
     )

package/package.json

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