Update Gas API to use MetaMask Gas API - Githubissues

MetaMask / metamask-extension

:globe_with_meridians: :electric_plug: The MetaMask browser extension enables browsing Ethereum blockchain enabled websites

https://metamask.io

Other

11.66k stars 4.78k forks source link

Update Gas API to use MetaMask Gas API #25194

Open desi opened 3 weeks ago

desi commented 3 weeks ago

What is this about?

We recently made a change to use a new Infura Gas API but we would like to go back to using the previous MetaMask Gas API which some additional changes are made to the Infura one. We should use gas.api.cx.metamask.io rather than gas.api.infura.io

dbrans commented 3 weeks ago

For context, here are the relevant changes:

Core gas-fee-controller PR #4068 – Released in gas-fee-controller@15.0.0
Extension PR #23717 – Extension is on gas-fee-controller@15.1.2 (both develop and production), hence the need for a hotfix.
Mobile PR #9693 – Mobile is on gas-fee-controller@13 in v7.24.0, so no urgency here. However, main is on gas-fee-controller@15.1.2.

Here's the plan for the hotfix:

Create a single extension PR to patch the gas-fee-controller and revert the changes. Merge the PR into develop.
Merge the PR into develop, cherry-pick it into 11.16.NEXT (the hotfix), and the Extension v12.0.0 RC.

After the hotfix, we'll need to regroup:

Revert MetaMask/core/pull/4068 in core on top of the latest gas-fee-controller (v17)
Upgrade gas-fee-controller in Extension and Mobile