@metamask-previews/network-controller 23.1.0-preview-bf2d2c4c → 23.1.0-preview-6cc620a

package/CHANGELOG.md

@@ -14,6 +14,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Update `NetworkClientConfiguration` so that `failoverUrls` is optional ([#5561](https://github.com/MetaMask/core/pull/5561))
   - This property was introduced in 23.0.0 as a breaking change, but this change makes it non-breaking
 
+### Fixed
+
+- Upgrade `@metamask/eth-json-rpc-infura` to `^10.1.1` and `@metamask/eth-json-rpc-infura` to `^16.0.1` ([#5573](https://github.com/MetaMask/core/pull/5573))
+  - This fixes a bug where non-standard unsuccessful JSON-RPC errors were being ignored/discarded
+
 ## [23.1.0]
 
 ### Added

package/dist/NetworkController.cjs

@@ -129,6 +129,10 @@ function getDefaultNetworkConfigurationsByChainId(additionalDefaultNetworks = []
 function getDefaultInfuraNetworkConfigurationsByChainId() {
     return Object.values(controller_utils_1.InfuraNetworkType).reduce((obj, infuraNetworkType) => {
         const chainId = controller_utils_1.ChainId[infuraNetworkType];
+        // Skip deprecated network as default network.
+        if (constants_1.DEPRECATED_NETWORKS.has(chainId)) {
+            return obj;
+        }
         const rpcEndpointUrl = 
         // This ESLint rule mistakenly produces an error.
         // eslint-disable-next-line @typescript-eslint/restrict-template-expressions
@@ -416,10 +420,8 @@ class NetworkController extends base_controller_1.BaseController {
         // ESLint is mistaken here; `name` is a string.
         // eslint-disable-next-line @typescript-eslint/restrict-template-expressions
         `${this.name}:getNetworkConfigurationByNetworkClientId`, this.getNetworkConfigurationByNetworkClientId.bind(this));
-        this.messagingSystem.registerActionHandler(
-        // TODO: Either fix this lint violation or explain why it's necessary to ignore.
-        // eslint-disable-next-line @typescript-eslint/restrict-template-expressions
-        `${this.name}:getSelectedNetworkClient`, this.getSelectedNetworkClient.bind(this));
+        this.messagingSystem.registerActionHandler(`${this.name}:getSelectedNetworkClient`, this.getSelectedNetworkClient.bind(this));
+        this.messagingSystem.registerActionHandler(`${this.name}:getSelectedChainId`, this.getSelectedChainId.bind(this));
         this.messagingSystem.registerActionHandler(
         // ESLint is mistaken here; `name` is a string.
         // eslint-disable-next-line @typescript-eslint/restrict-template-expressions
@@ -458,6 +460,15 @@ class NetworkController extends base_controller_1.BaseController {
         }
         return undefined;
     }
+    /**
+     * Accesses the chain ID from the selected network client.
+     *
+     * @returns The chain ID of the selected network client in hex format or undefined if there is no network client.
+     */
+    getSelectedChainId() {
+        const networkConfiguration = this.getNetworkConfigurationByNetworkClientId(this.state.selectedNetworkClientId);
+        return networkConfiguration?.chainId;
+    }
     /**
      * Internally, the Infura and custom network clients are categorized by type
      * so that when accessing either kind of network client, TypeScript knows