chainChanged emits on page load

Describe the bug Sometimes chainChanged message emits on every page load. So if I have this line in my code: ethereum.on('chainChanged', (_chainId) => window.location.reload()); then user gets endless page refresh loop.

To Reproduce I couldn’t reproduce this, but one of our user shared his screen and I saw this endless page refresh loop. I added console.log('chainChanged'); to that event callback, and console showed this log every time before page refreshed.

Expected behavior chainChanged event should be sent only on chain change, not on page load

Browser details

• OS: OS X
• Browser: last Chrome
• MetaMask Version: 8.0.8

[mqklin]

@mehranhydary to fix this I store lastChainId in localStorage and on chainChained check if it’s the same:

      (async () => {
        try {
          const chainId = await web3.eth.getChainId();
          StoredLastEthereumChain.set(`0x${chainId.toString(16)}`); // save to localStorage

          web3.currentProvider.on(
            'chainChanged',
            chainId => {
              const lastChainId = StoredLastEthereumChain.get(); // get from localStorage
              if (String(chainId) !== String(lastChainId)) {
                window.location.reload();
              }
            },
          );
        }
        catch (e) {
          // handle exception
        }
      })();

[laygir]

I’m having this issue with Brave but not with Chrome.

Brave v1.34.81 Chromium: 97.0.4692.99 MetaMask v10.8.1