Skip to content

chore: bump network-controller preview to force failover build#32098

Draft
cryptodev-2s wants to merge 1 commit into
fb/bump-network-controllerfrom
feat/force-rpc-failover
Draft

chore: bump network-controller preview to force failover build#32098
cryptodev-2s wants to merge 1 commit into
fb/bump-network-controllerfrom
feat/force-rpc-failover

Conversation

@cryptodev-2s

Copy link
Copy Markdown
Contributor

Description

Bumps the @metamask/network-controller preview build to 32.0.0-preview-eca6f2051, which adds the forced RPC failover capability.

When the core-platform-rpc-failover-force-enabled remote feature flag is on, any Infura endpoint that has failover URLs routes all of its traffic (including block tracker polling) to those failover URLs and never contacts Infura. It is an emergency kill switch for cases where the automatic failover logic itself misbehaves, and it works independently of the existing walletFrameworkRpcFailoverEnabled flag.

No app code changes are needed: network-controller reads the flag itself through RemoteFeatureFlagController, the networks already carry failover URLs, and the change is non breaking. This PR is just the dependency bump so the capability is available once the LaunchDarkly flag is created and enabled.

Core PR: MetaMask/core#9175

Changelog

CHANGELOG entry: null

Related issues

Fixes: https://consensyssoftware.atlassian.net/browse/WPC-1112

Manual testing steps

Feature: Forced RPC failover

  Scenario: forced failover routes Infura traffic to the failover endpoint
    Given the core-platform-rpc-failover-force-enabled remote flag is on
    And the selected network is an Infura network with failover URLs configured
    When the app makes RPC requests on that network
    Then the requests go to the failover endpoint and not to Infura

  Scenario: turning the flag off restores Infura
    Given the core-platform-rpc-failover-force-enabled remote flag is off
    When the app makes RPC requests on an Infura network
    Then the requests go to Infura as usual

Screenshots/Recordings

N/A, dependency bump with no UI changes.

Before

N/A

After

N/A

Pre-merge author checklist

Performance checks (if applicable)

  • I've tested on Android
    • Ideally on a mid-range device; emulator is acceptable
  • I've tested with a power user scenario
    • Use these power-user SRPs to import wallets with many accounts and tokens
  • I've instrumented key operations with Sentry traces for production performance metrics

For performance guidelines and tooling, see the Performance Guide.

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@github-actions

Copy link
Copy Markdown
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@github-actions github-actions Bot added the pr-not-ready-for-e2e Skip E2E and block merging. Remove this label once the PR is ready to run the E2E tests. label Jun 19, 2026
@mm-token-exchange-service mm-token-exchange-service Bot added the team-core-platform Core Platform team label Jun 19, 2026
@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatednpm/​@​metamask/​network-controller@​32.0.0-preview-22ab7e3 ⏵ 32.0.0-preview-eca6f205100000

View full report

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-ready-for-e2e Skip E2E and block merging. Remove this label once the PR is ready to run the E2E tests. size-XS team-core-platform Core Platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant