Revert if a call fails in an ERC2771Forwarder atomic batch

.changeset/some-dolls-shine.md

@@ -0,0 +1,5 @@
+---
+'openzeppelin-solidity': minor
+---
+
+`ERC2771Forwarder`: Revert the entire atomic batch if one of the calls fails.

contracts/metatx/ERC2771Forwarder.sol

@@ -75,6 +75,11 @@ contract ERC2771Forwarder is EIP712, Nonces {
      */
     event ExecutedForwardRequest(address indexed signer, uint256 nonce, bool success);
 
+    /**
+     * @dev One of the calls in an atomic batch failed.
+     */
+    error ERC2771ForwarderFailureInAtomicBatch();
+
     /**
      * @dev The request `from` doesn't match with the recovered `signer`.
      */
@@ -173,6 +178,7 @@ contract ERC2771Forwarder is EIP712, Nonces {
             requestsValue += requests[i].value;
             bool success = _execute(requests[i], atomic);
             if (!success) {
+                if (atomic) revert ERC2771ForwarderFailureInAtomicBatch();
                 refundValue += requests[i].value;
             }
         }

test/metatx/ERC2771Forwarder.test.js

@@ -227,6 +227,20 @@ describe('ERC2771Forwarder', function () {
           expect(await this.forwarder.nonces(request.from)).to.equal(request.nonce + 1n);
         }
       });
+
+      it('atomic batch with reverting request reverts the whole batch', async function () {
+        // Add extra reverting request
+        await this.forgeRequest(
+          { value: 10n, data: this.receiver.interface.encodeFunctionData('mockFunctionRevertsNoReason') },
+          this.accounts[requestCount],
+        ).then(extraRequest => this.requests.push(extraRequest));
+        // recompute total value with the extra request
+        this.value = requestsValue(this.requests);
+
+        await expect(
+          this.forwarder.executeBatch(this.requests, ethers.ZeroAddress, { value: this.value }),
+        ).to.be.revertedWithCustomError(this.forwarder, 'ERC2771ForwarderFailureInAtomicBatch');
+      });
     });
 
     describe('with tampered requests', function () {