From e6d1b052f024f174aa5be06363334d24136f3073 Mon Sep 17 00:00:00 2001 From: Mark Stacey Date: Fri, 14 Apr 2023 19:10:32 -0230 Subject: [PATCH] Make `_setProviderConfig` async (#18600) The network controller internal method `_setProviderConfig` has been made async, the async `_switchNetwork` operation is now `await`-ed. Since the `_switchNetwork` call was the last operation, this has zero functional impact. Relates to https://github.com/MetaMask/metamask-extension/issues/18587 --- app/scripts/controllers/network/network-controller.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/scripts/controllers/network/network-controller.ts b/app/scripts/controllers/network/network-controller.ts index a75cd6100..ee0ea686c 100644 --- a/app/scripts/controllers/network/network-controller.ts +++ b/app/scripts/controllers/network/network-controller.ts @@ -866,10 +866,10 @@ export class NetworkController extends EventEmitter { * * @param providerConfig - The provider configuration. */ - _setProviderConfig(providerConfig: ProviderConfiguration): void { + async _setProviderConfig(providerConfig: ProviderConfiguration) { this.#previousProviderConfig = this.providerStore.getState(); this.providerStore.putState(providerConfig); - this._switchNetwork(providerConfig); + await this._switchNetwork(providerConfig); } /**