Test accessLists with modern MetaMask

[ricmoo]

See #1876

The JsonRpcSigner may be passing the accessList incorrectly to MetaMask. It looks a bit strange and doesn’t seem to be working and now MetaMask has accessList support so it can be tested.

[novaknole]

Hi @ricmoo ,

Any news on this ?

[ricmoo]

I’ve been configuring a few things so I can test this out. I will have more info later today. Sorry for the delay.

[ricmoo]

I've replicated this and believe it to be a bug in ethers. Trying to track it down now...

[ricmoo]

I have opened an issue with MetaMask too (see MetaMask/metamask-extension#11863), as I can replicate it in ethers, but have also added debugging that leads me to believe it might be on their side.

We're Working on it. :)

[ricmoo]

Update: looks like MetaMask has confirmed it is an issue on their side.

[novaknole]

@ricmoo

Thanks so much for the follow ups and figuring this out. I appreciate it so much.

It's really a bummer for me. Could I ask for your advice here ?

// ContractA
function nice(address to) public payable {
     require(_to.send(_value), ERROR_SEND_REVERTED);
}

I also have a contractB which was deployed with EIP-1167. Now, The problem is that when I call contractA's nice function and pass contractB address, it fails.

The reason it fails is that contractB is a proxy contract, so it needs to call delegateCall first, and delegateCall costs 2600 Gas Cost which is more than 2300 Gas Cost for send. This is why I was using accessList to warm the addresses and pay for them while starting a transaction.

Is there any other choice/way to solve this ? maybe you know if some other wallets support this or anything that might help a little bit ?

Thanks a lot.

[novaknole]

@ricmoo

Hey Richard,

any ideas on my last question ?

[novaknole]

or maybe somehow, you know which wallet supports it and you tested it !!!

[ricmoo]

You don't really have any options other than using a wallet that supports it. That could mean using an ethers.js Wallet object, or local Geth node with private keys. Those should both work fine.

I've checked with MyCrypto, and they also don't support it. I've only tested with Geth, as there are no other options I've found so far to test with...

[ricmoo]

I’m going to move this to discussions, as it is not an issue with ethers and might be useful to further discuss. :)