braid-http 1.3.23 → 1.3.24

package/braid-http-client.js

@@ -853,13 +853,17 @@ async function multiplex_fetch(url, params) {
             // attempt to establish a multiplexed connection
             try {
                 if (params.use_multiplex_header) throw 'skip to trying header'
-                var r = await braid_fetch(`${origin}/${multiplexer}`, {method: 'MULTIPLEX', retry: true})
+                var r = await braid_fetch(`${origin}/${multiplexer}`, {method: 'MULTIPLEX', headers: {'Multiplex-Version': '0.0.1'}, retry: true})
+                if (!r.ok || r.headers.get('Multiplex-Version') !== '0.0.1') throw 'bad'
             } catch (e) {
                 // some servers don't like custom methods,
                 // so let's try with a custom header
                 try {
                     using_multiplex_header = true
-                    r = await braid_fetch(`${origin}/${multiplexer}`, {headers: {MULTIPLEX: true}, retry: true})
+                    r = await braid_fetch(`${origin}/${multiplexer}`, {headers: {Multiplex: true, 'Multiplex-Version': '0.0.1'}, retry: true})
+
+                    if (!r.ok) throw new Error('status not ok: ' + r.status)
+                    if (r.headers.get('Multiplex-Version') !== '0.0.1') throw new Error('wrong multiplex version: ' + r.headers.get('Multiplex-Version') + ', expected 0.0.1')
                 } catch (e) {
                     // fallback to normal fetch if multiplexed connection fails
                     console.error(`Could not establish multiplexed connection.\nGot error: ${e}.\nFalling back to normal connection.`)
@@ -895,7 +899,8 @@ async function multiplex_fetch(url, params) {
 
             // add the multiplexer header without affecting the underlying params
             var mux_headers = new Headers(params.headers)
-            mux_headers.set('multiplexer', `/${multiplexer}/${stream}`)
+            mux_headers.set('Multiplexer', `/${multiplexer}/${stream}`)
+            mux_headers.set('Multiplex-Version', '0.0.1')
             params = {...params, headers: mux_headers}
 
             // setup a way to receive incoming data from the multiplexer
@@ -933,9 +938,18 @@ async function multiplex_fetch(url, params) {
                 stream_error = e
                 bytes_available()
                 try {
-                    await braid_fetch(`${origin}${params.headers.get('multiplexer')}`, {...using_multiplex_header ? {headers: {MULTIPLEX: true}} : {method: 'MULTIPLEX'}, retry: true})
+                    var r = await braid_fetch(`${origin}${params.headers.get('multiplexer')}`, {
+                        ...!using_multiplex_header && {method: 'MULTIPLEX'},
+                        headers: {
+                            ...using_multiplex_header && {Multiplex: true},
+                            'Multiplex-Version': '0.0.1'
+                        }, retry: true})
+
+                    if (!r.ok) throw new Error('status not ok: ' + r.status)
+                    if (r.headers.get('Multiplex-Version') !== '0.0.1') throw new Error('wrong multiplex version: ' + r.headers.get('Multiplex-Version') + ', expected 0.0.1')
                 } catch (e) {
-                    console.error(`Could not cancel multiplexed connection:`, e)
+                    e = new Error(`Could not cancel multiplexed connection: ${e}`)
+                    console.error('' + e)
                     throw e
                 }
             }
@@ -951,7 +965,9 @@ async function multiplex_fetch(url, params) {
                     throw new Error('multiplexer not yet connected')
                 }
 
-                if (res.status !== 293) throw new Error('Could not establish multiplexed stream ' + params.headers.get('multiplexer') + ' got status: ' + res.status)
+                if (res.status !== 293) throw new Error('Could not establish multiplexed stream ' + params.headers.get('multiplexer') + ', got status: ' + res.status)
+
+                if (res.headers.get('Multiplex-Version') !== '0.0.1') throw new Error('Could not establish multiplexed stream ' + params.headers.get('multiplexer') + ', got unknown version: ' + res.headers.get('Multiplex-Version'))
 
                 // we want to present the illusion that the connection is still open,
                 // and therefor closable with "abort",

package/braid-http-server.js

@@ -243,7 +243,10 @@ function braidify (req, res, next) {
     req.subscribe = subscribe
 
     // Multiplexer stuff
-    if (braidify.use_multiplexing && (req.method === 'MULTIPLEX' || req.headers.multiplex)) {
+    if (braidify.use_multiplexing &&
+        (req.method === 'MULTIPLEX' || req.headers.multiplex) &&
+        req.headers['multiplex-version'] === '0.0.1') {
+
         // parse the multiplexer id and stream id from the url
         var [multiplexer, stream] = req.url.slice(1).split('/')
 
@@ -263,6 +266,7 @@ function braidify (req, res, next) {
             // keep the connection open,
             // so people can send multiplexed data to it
             res.writeHead(200, {
+                'Multiplex-Version': '0.0.1',
                 'Cache-Control': 'no-cache',
                 'X-Accel-Buffering': 'no',
                 ...req.httpVersion !== '2.0' && {'Connection': 'keep-alive'}
@@ -290,8 +294,11 @@ function braidify (req, res, next) {
                 m.streams.delete(stream)
             } else m.streams.set(stream, 'abort')
 
+            console.log('got here....!!!!')
+
+
             // let the requester know we succeeded
-            res.writeHead(200, {})
+            res.writeHead(200, { 'Multiplex-Version': '0.0.1' })
             return res.end(``)
         }
     }
@@ -299,7 +306,10 @@ function braidify (req, res, next) {
     // a multiplexer header means the user wants to send the
     // results of this request to the provided multiplexer,
     // tagged with the given stream id
-    if (braidify.use_multiplexing && req.headers.multiplexer) {
+    if (braidify.use_multiplexing &&
+        req.headers.multiplexer &&
+        req.headers['multiplex-version'] === '0.0.1') {
+
         // parse the multiplexer id and stream id from the url
         var [multiplexer, stream] = req.headers.multiplexer.slice(1).split('/')
 
@@ -319,7 +329,10 @@ function braidify (req, res, next) {
         }
 
         // let the requester know we've multiplexed their response
-        res.writeHead(293, {multiplexer: req.headers.multiplexer})
+        res.writeHead(293, {
+            multiplexer: req.headers.multiplexer,
+            'Multiplex-Version': '0.0.1'
+        })
         res.end('Ok.')
 
         // and now set things up so that future use of the

package/package.json

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