Conversation
|
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. |
0d425cb to
dd9b20b
Compare
|
@metamaskbot update-policies |
|
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@metamask/network-controller@18.1.2, npm/@metamask/permission-controller@9.1.0 |
SelectedNetworkController to fix issue where selected network domain is incorrectly set
|
Policies updated |
|
@metamaskbot update-policies |
|
Policies updated |
c6c2a49 to
2267ae3
Compare
|
👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎ This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
|
@metamaskbot update-policies |
|
Policies updated |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #25106 +/- ##
========================================
Coverage 65.62% 65.62%
========================================
Files 1372 1372
Lines 54512 54512
Branches 14281 14281
========================================
Hits 35769 35769
Misses 18743 18743 ☔ View full report in Codecov by Sentry. |
Builds ready [21820a9]
Page Load Metrics (148 ± 200 ms)
|
Description
Pulls this fix which adds a guard to the
setNetworkClientIdForDomainmethod on theSelectedNetworkControllerto no longer add domains to domains state (nor create a selected network proxy for them) unless theuseRequestQueuePreferenceflag is trueBumping the
@metamask/selected-network-controllerversion required bumping peer dependencies:@metamask/network-controllerto v19.0.0@metamask/permission-controllerto v10.0.0- This required bumping its own peerDep
@metamask/approval-controllerto v7.0.0Related issues
See: https://consensys.slack.com/archives/C1L7H42BT/p1717512150509719
Manual testing steps
Select networks for each sitesettingeth_chainIdscriptand connect an account to the site
9. Repeat steps 1 - 7
10. The
eth_chainIdresults should still match the globally selected networkScreenshots/Recordings
Before
Screen.Recording.2024-06-06.at.12.16.28.PM.mov
After
Screen.Recording.2024-06-06.at.11.35.35.AM.mov
Pre-merge author checklist
Pre-merge reviewer checklist