Resolves #1837

There were two finally's that returned promises. They used to work before because of a faulty implementation of Promise.finally. When that faulty implementation was corrected in 4.0.1-beta.2, the faulty code started to lock tables endlessly in certain scenarios.

addons/dexie-cloud/src/middleware-helpers/guardedTable.ts

@@ -30,7 +30,7 @@ function readLock<TReq extends { trans: DBCoreTransaction }, TRes>(
     const promise = (numWriters > 0
       ? writers[numWriters - 1].then(() => fn(req), () => fn(req))
       : fn(req)
-    ).finally(() => readers.splice(readers.indexOf(promise)));
+    ).finally(() => {readers.splice(readers.indexOf(promise))});
     readers.push(promise);
     return promise;
   };
@@ -51,7 +51,7 @@ function writeLock<TReq extends { trans: DBCoreTransaction }, TRes>(
       : readers.length > 0
       ? allSettled(readers).then(() => fn(req))
       : fn(req)
-    ).finally(() => writers.shift());
+    ).finally(() => {writers.shift();});
     writers.push(promise);
     return promise;
   };