indexeddbshim 17.2.0 → 17.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "indexeddbshim",
-  "version": "17.2.0",
+  "version": "17.2.1",
   "type": "module",
   "author": "Parashuram <code@nparashuram.com>",
   "contributors": [
@@ -21,8 +21,8 @@
       "web-platform-tests/**",
       "coverage",
       "ignore",
-      ".eslintrc.js",
-      ".mocharc.cjs",
+      "eslint.config.js",
+      ".mocharc.js",
       "tests-mocha/**",
       "tests-polyfill/**",
       "test-support/**"
@@ -210,7 +210,6 @@
     "rollup-plugin-node-polyfills": "^0.2.1",
     "sinon": "^22.1.0",
     "source-map-support": "0.5.21",
-    "sourcemap-transformer": "git+https://github.com/brettz9/sourcemap-transformer.git",
     "typescript": "^6.0.3",
     "w3c-blob": "0.0.1",
     "ws": "^8.21.3"

package/src/IDBFactory.js

@@ -856,9 +856,17 @@ IDBFactory.prototype.deleteDatabase = function (name) {
      * @returns {boolean}
      */
     function dbError (tx, err) {
-        if (calledDBError || err === true) {
+        if (calledDBError) {
             return false;
         }
+        // Must be set synchronously (not inside the `sysdbFinishedCbDelete` callback below) to
+        //  guard against reentrancy: `sysdbFinishedCbDelete` -> `rollback`/`commit` -> websql-configurable's
+        //  `done()` -> `currentTask.errorCallback(er)`, which is this very `dbError`, calling back into
+        //  us before the first invocation has finished. Without this, a second rollback/commit gets
+        //  issued on the shared `sysdb` connection after it has already moved on to the next queued
+        //  transaction, corrupting that connection's task queue (see issue with unrelated `open()` calls
+        //  failing/crashing after a `deleteDatabase` file-removal error).
+        calledDBError = true;
         const er = webSQLErrback(/** @type {SQLError} */ (err || tx));
         sysdbFinishedCbDelete(true, function () {
             req.__done = true;
@@ -867,7 +875,6 @@ IDBFactory.prototype.deleteDatabase = function (name) {
             // Re: why bubbling here (and how cancelable is only really relevant for `window.onerror`) see: https://github.com/w3c/IndexedDB/issues/86
             const e = createEvent('error', er, {bubbles: true, cancelable: true});
             req.dispatchEvent(e);
-            calledDBError = true;
         });
         return false;
     }