fix: Make transaction rollback best effort. (#1967)

tom-andersen · web-flow · commit 1d76546a7815 · 2024-01-05T16:21:37.000-05:00
* Make rollback best effort.

* Fix test
diff --git a/dev/src/transaction.ts b/dev/src/transaction.ts
@@ -481,7 +481,20 @@ export class Transaction implements firestore.Transaction {
       transaction: this._transactionId,
     };
 
-    return this._firestore.request('rollback', request, this._requestTag);
+    const promise: Promise<void> = this._firestore.request(
+      'rollback',
+      request,
+      this._requestTag
+    );
+
+    return promise.catch(reason => {
+      logger(
+        'Firestore.runTransaction',
+        this._requestTag,
+        'Best effort to rollback failed with error:',
+        reason
+      );
+    });
   }
 
   /**
diff --git a/dev/test/transaction.ts b/dev/test/transaction.ts
@@ -574,7 +574,6 @@ describe('failed transactions', () => {
           begin({transactionId: 'foo1'}),
           commit('foo1', /* writes=*/ undefined, serverError),
           rollback('foo1', serverError),
-          rollback('foo1'),
           backoff(),
           begin({
             transactionId: 'foo2',
