braid-http 0.2.0 → 0.3.1

package/braid-http-client.js

@@ -154,8 +154,8 @@ async function braid_fetch (url, params = {}) {
 
     // We provide some shortcuts for Braid params
     if (params.version)
-        params.headers.set('version', JSON.stringify(params.version))
-    if (params.parents)
+        params.headers.set('version', params.version.map(JSON.stringify).join(', '))
+    if (params.parents?.length)
         params.headers.set('parents', params.parents.map(JSON.stringify).join(', '))
     if (params.subscribe)
         params.headers.set('subscribe', 'true')
@@ -487,9 +487,8 @@ function parse_headers (input) {
 
     // Success!  Let's parse special headers
     if ('version' in headers)
-        headers.version = JSON.parse(headers.version)
-    if ('parents' in headers)
-        headers.parents = JSON.parse('['+headers.parents+']')
+        headers.version = JSON.parse('['+headers.version+']')
+    headers.parents = JSON.parse('['+(headers.parents ?? '')+']')
     if ('patches' in headers)
         headers.patches = JSON.parse(headers.patches)
 

package/braid-http-server.js

@@ -195,8 +195,8 @@ function braidify (req, res, next) {
     res.setHeader('Range-Request-Allow-Units', 'json')
 
     // Extract braid info from headers
-    var version = req.headers.version && JSON.parse(req.headers.version),
-        parents = req.headers.parents && JSON.parse('['+req.headers.parents+']'),
+    var version = JSON.parse('['+(req.headers.version ?? '')+']'),
+        parents = JSON.parse('['+(req.headers.parents ?? '')+']'),
         peer = req.headers['peer'],
         url = req.url.substr(1)
 
@@ -335,10 +335,11 @@ function send_version(res, data, url, peer) {
         // Version and Parents get output in the Structured Headers format
         if (header === 'version') {
             header = 'Version'               // Capitalize for prettiness
-            value = JSON.stringify(value)
+            value = value.map(JSON.stringify).join(", ")
         } else if (header === 'parents') {
             header = 'Parents'               // Capitalize for prettiness
-            value = parents.map(JSON.stringify).join(", ")
+            if (value.length == 0) continue  // we express no parents as not having the field at all
+            value = value.map(JSON.stringify).join(", ")
         }
 
         // We don't output patches or body yet

package/package.json

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

package/readme.md

@@ -1,6 +1,6 @@
 # Braid-HTTP
 
-This polyfill library implements the [Braid-HTTP v03 protocol](https://github.com/braid-org/braid-spec/blob/master/draft-toomim-httpbis-braid-http-03.txt) in Javascript.  It gives browsers a `braid_fetch()` drop-in replacement for the `fetch()` API, and gives nodejs an `http` plugin, allowing them to speak Braid in a simple way.
+This polyfill library implements the [Braid-HTTP v04 protocol](https://github.com/braid-org/braid-spec/blob/master/draft-toomim-httpbis-braid-http-04.txt) in Javascript.  It gives browsers a `braid_fetch()` drop-in replacement for the `fetch()` API, and gives nodejs an `http` plugin, allowing them to speak Braid in a simple way.
 
 Developed in [braid.org](https://braid.org).