braid-text 0.2.20 → 0.2.22

package/index.js

@@ -26,8 +26,8 @@ braid_text.serve = async (req, res, options = {}) => {
 
     braid_text.free_cors(res)
 
-    function my_end(statusCode, x) {
-        res.statusCode = statusCode
+    function my_end(statusCode, x, statusText, headers) {
+        res.writeHead(statusCode, statusText, headers)
         res.end(x ?? '')
     }
 
@@ -134,10 +134,10 @@ braid_text.serve = async (req, res, options = {}) => {
         let done_my_turn = null
         prev_put_p = new Promise(
             (done) =>
-            (done_my_turn = (statusCode, x) => {
+            (done_my_turn = (statusCode, x, statusText, headers) => {
                 waiting_puts--
                 if (braid_text.verbose) console.log(`waiting_puts(after--) = ${waiting_puts}`)
-                my_end(statusCode, x)
+                my_end(statusCode, x, statusText, headers)
                 done()
             })
         )
@@ -190,7 +190,7 @@ braid_text.serve = async (req, res, options = {}) => {
                 // - 428 Precondition Required
                 //     - pros: the name sounds right
                 //     - cons: typically implies that the request was missing an http conditional field like If-Match. that is to say, it implies that the request is missing a precondition, not that the server is missing a precondition
-                return done_my_turn(425, e.message)
+                return done_my_turn(400, e.message, 'Missing Parents', { 'Retry-After': '1' })
             } else {
                 return done_my_turn(500, "The server failed to apply this version. The error generated was: " + e)
             }

package/package.json

@@ -1,12 +1,12 @@
 {
   "name": "braid-text",
-  "version": "0.2.20",
+  "version": "0.2.22",
   "description": "Library for collaborative text over http using braid.",
   "author": "Braid Working Group",
   "repository": "braid-org/braid-text",
   "homepage": "https://braid.org",
   "dependencies": {
     "diamond-types-node": "^1.0.2",
-    "braid-http": "^1.3.73"
+    "braid-http": "^1.3.74"
   }
 }

package/test/test.html

@@ -238,4 +238,32 @@ runTest(
     '19'
 )
 
+runTest(
+    "test retry when parents not there..",
+    async () => {
+        return await new Promise(done => {
+            var count = 0
+            var key = 'test-' + Math.random().toString(36).slice(2)
+            var a = new AbortController()
+            braid_fetch(`/${key}`, {
+                signal: a.signal,
+                multiplex: false,
+                method: 'PUT',
+                version: ['hi-3'],
+                parents: ['hi-1'],
+                body: 'xx',
+                onFetch: () => {
+                    count++
+                    if (count === 2) {
+                        done('retried!')
+                        a.abort()
+                    }
+                },
+                retry: true
+            })
+        })
+    },
+    'retried!'
+)
+
 </script>