Uncaught aggregator error when submitting a bundle with invalid contract address

Steps to Reproduce

Construct a bundle where one of the actions has a contractAddress of 0x.

const bundle = wallet.sign({
    nonce: await wallet.Nonce(),
    actions: [{
      ethValue: ethers.utils.parseEther("1"),
      contractAddress: "0x",
      encodedFunction: "0x",
    }],
  });

Then submit the bundle to an aggregator

Expected Result

• Aggregator rejects the bundle for being invalid (HTTP 400) or
• Aggregator fails bundle and does not include it

Actual result

Aggregator crashes due to uncaught exception. This persists between restarts

Severity

A malicious or unknowing user could DoS an aggregator instance(s)

[jacque006]

@thearyanag done, let us know if you have any questions or need help working on this. Thanks for picking it up!

[jacque006]

High level overview: https://github.com/web3well/bls-wallet/blob/main/docs/system_overview.md Local repo/project setup instructions: https://github.com/web3well/bls-wallet/blob/main/docs/local_development.md Example of how to send a transaction: https://github.com/web3well/bls-wallet/blob/main/docs/use_bls_wallet_clients.md#send-a-transaction

Main thing to reproduce is setting contractAddress: "0x" in an action and submitting to an aggregator. The fix will likely be in aggregator code or in client.