-
Notifications
You must be signed in to change notification settings - Fork 5.4k
fix: existing EVM permissions removed when a Solana Wallet Standard connection is rejected #36283
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
@metamaskbot update-policies |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
No policy changes |
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [6e6800b]
UI Startup Metrics (1217 ± 79 ms)
Benchmark value 26 exceeds gate value 23 for chrome browserify home mean firstReactRender Benchmark value 5 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 831 exceeds gate value 830 for chrome browserify home mean loadScripts Benchmark value 1400 exceeds gate value 1365 for chrome browserify home p95 uiStartup Benchmark value 1240 exceeds gate value 1190 for chrome browserify home p95 load Benchmark value 1231 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded Benchmark value 228 exceeds gate value 18 for chrome browserify home p95 backgroundConnect Benchmark value 11 exceeds gate value 1.2 for chrome browserify home p95 initialActions Benchmark value 1024 exceeds gate value 940 for chrome browserify home p95 loadScripts Benchmark value 13 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 1536 exceeds gate value 1405 for firefox browserify home mean uiStartup Benchmark value 1329 exceeds gate value 1245 for firefox browserify home mean load Benchmark value 1329 exceeds gate value 1239 for firefox browserify home mean domContentLoaded Benchmark value 37 exceeds gate value 25 for firefox browserify home mean backgroundConnect Benchmark value 33 exceeds gate value 25 for firefox browserify home mean firstReactRender Benchmark value 4 exceeds gate value 1 for firefox browserify home mean initialActions Benchmark value 1301 exceeds gate value 1230 for firefox browserify home mean loadScripts Benchmark value 11 exceeds gate value 9 for firefox browserify home mean setupStore Benchmark value 1737 exceeds gate value 1660 for firefox browserify home p95 uiStartup Benchmark value 233 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 6 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 1703 exceeds gate value 1615 for firefox webpack home mean uiStartup Benchmark value 1441 exceeds gate value 1380 for firefox webpack home mean load Benchmark value 1441 exceeds gate value 1380 for firefox webpack home mean domContentLoaded Benchmark value 121 exceeds gate value 100 for firefox webpack home mean domInteractive Benchmark value 35 exceeds gate value 26 for firefox webpack home mean backgroundConnect Benchmark value 49 exceeds gate value 38 for firefox webpack home mean firstReactRender Benchmark value 6 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 1416 exceeds gate value 1360 for firefox webpack home mean loadScripts Benchmark value 2079 exceeds gate value 1935 for firefox webpack home p95 uiStartup Benchmark value 1703 exceeds gate value 1660 for firefox webpack home p95 load Benchmark value 1702 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded Benchmark value 299 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 63 exceeds gate value 50 for firefox webpack home p95 firstReactRender Benchmark value 11 exceeds gate value 2 for firefox webpack home p95 initialActions Benchmark value 1680 exceeds gate value 1630 for firefox webpack home p95 loadScripts Sum of mean exceeds: 926ms | Sum of p95 exceeds: 1008.8ms Sum of all benchmark exceeds: 1934.8ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
wenfix
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [e4ab92d]
UI Startup Metrics (1216 ± 72 ms)
Benchmark value 26 exceeds gate value 23 for chrome browserify home mean firstReactRender Benchmark value 5 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 836 exceeds gate value 830 for chrome browserify home mean loadScripts Benchmark value 222 exceeds gate value 18 for chrome browserify home p95 backgroundConnect Benchmark value 11 exceeds gate value 1.2 for chrome browserify home p95 initialActions Benchmark value 964 exceeds gate value 940 for chrome browserify home p95 loadScripts Benchmark value 15 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 30 exceeds gate value 25 for firefox browserify home mean backgroundConnect Benchmark value 30 exceeds gate value 25 for firefox browserify home mean firstReactRender Benchmark value 5 exceeds gate value 1 for firefox browserify home mean initialActions Benchmark value 12 exceeds gate value 9 for firefox browserify home mean setupStore Benchmark value 19 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 35 exceeds gate value 27 for firefox browserify home p95 setupStore Benchmark value 1664 exceeds gate value 1615 for firefox webpack home mean uiStartup Benchmark value 1420 exceeds gate value 1380 for firefox webpack home mean load Benchmark value 1420 exceeds gate value 1380 for firefox webpack home mean domContentLoaded Benchmark value 112 exceeds gate value 100 for firefox webpack home mean domInteractive Benchmark value 30 exceeds gate value 26 for firefox webpack home mean backgroundConnect Benchmark value 49 exceeds gate value 38 for firefox webpack home mean firstReactRender Benchmark value 6 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 1398 exceeds gate value 1360 for firefox webpack home mean loadScripts Benchmark value 310 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 61 exceeds gate value 50 for firefox webpack home p95 firstReactRender Benchmark value 9 exceeds gate value 2 for firefox webpack home p95 initialActions Benchmark value 29 exceeds gate value 28 for firefox webpack home p95 setupStore Sum of mean exceeds: 433ms | Sum of p95 exceeds: 443.8ms Sum of all benchmark exceeds: 876.8ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [27b6c37]
UI Startup Metrics (1189 ± 60 ms)
Benchmark value 25 exceeds gate value 23 for chrome browserify home mean firstReactRender Benchmark value 6 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 220 exceeds gate value 18 for chrome browserify home p95 backgroundConnect Benchmark value 13 exceeds gate value 1.2 for chrome browserify home p95 initialActions Benchmark value 18 exceeds gate value 17 for chrome browserify home p95 setupStore Benchmark value 14 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 1450 exceeds gate value 1405 for firefox browserify home mean uiStartup Benchmark value 1241 exceeds gate value 1239 for firefox browserify home mean domContentLoaded Benchmark value 36 exceeds gate value 25 for firefox browserify home mean backgroundConnect Benchmark value 30 exceeds gate value 25 for firefox browserify home mean firstReactRender Benchmark value 8 exceeds gate value 1 for firefox browserify home mean initialActions Benchmark value 15 exceeds gate value 9 for firefox browserify home mean setupStore Benchmark value 1821 exceeds gate value 1660 for firefox browserify home p95 uiStartup Benchmark value 245 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 89 exceeds gate value 70 for firefox browserify home p95 backgroundConnect Benchmark value 25 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 39 exceeds gate value 27 for firefox browserify home p95 setupStore Benchmark value 1621 exceeds gate value 1615 for firefox webpack home mean uiStartup Benchmark value 1382 exceeds gate value 1380 for firefox webpack home mean load Benchmark value 1382 exceeds gate value 1380 for firefox webpack home mean domContentLoaded Benchmark value 108 exceeds gate value 100 for firefox webpack home mean domInteractive Benchmark value 32 exceeds gate value 26 for firefox webpack home mean backgroundConnect Benchmark value 48 exceeds gate value 38 for firefox webpack home mean firstReactRender Benchmark value 8 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 275 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 57 exceeds gate value 50 for firefox webpack home p95 firstReactRender Benchmark value 10 exceeds gate value 2 for firefox webpack home p95 initialActions Sum of mean exceeds: 326ms | Sum of p95 exceeds: 620.8ms Sum of all benchmark exceeds: 946.8ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
|
❌ test-e2e-chrome-api-specs failed. View the html report here. |
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [23df428]
UI Startup Metrics (1241 ± 58 ms)
Benchmark value 1075 exceeds gate value 1070 for chrome browserify home mean load Benchmark value 1067 exceeds gate value 1061 for chrome browserify home mean domContentLoaded Benchmark value 256 exceeds gate value 10 for chrome browserify home mean backgroundConnect Benchmark value 26 exceeds gate value 23 for chrome browserify home mean firstReactRender Benchmark value 6 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 272 exceeds gate value 18 for chrome browserify home p95 backgroundConnect Benchmark value 16 exceeds gate value 1.2 for chrome browserify home p95 initialActions Benchmark value 18 exceeds gate value 17 for chrome browserify home p95 setupStore Benchmark value 15 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 1450 exceeds gate value 1405 for firefox browserify home mean uiStartup Benchmark value 1248 exceeds gate value 1245 for firefox browserify home mean load Benchmark value 1248 exceeds gate value 1239 for firefox browserify home mean domContentLoaded Benchmark value 34 exceeds gate value 25 for firefox browserify home mean backgroundConnect Benchmark value 29 exceeds gate value 25 for firefox browserify home mean firstReactRender Benchmark value 8 exceeds gate value 1 for firefox browserify home mean initialActions Benchmark value 12 exceeds gate value 9 for firefox browserify home mean setupStore Benchmark value 260 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 12 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 48 exceeds gate value 27 for firefox browserify home p95 setupStore Benchmark value 108 exceeds gate value 100 for firefox webpack home mean domInteractive Benchmark value 31 exceeds gate value 26 for firefox webpack home mean backgroundConnect Benchmark value 47 exceeds gate value 38 for firefox webpack home mean firstReactRender Benchmark value 6 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 285 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 58 exceeds gate value 50 for firefox webpack home p95 firstReactRender Benchmark value 10 exceeds gate value 2 for firefox webpack home p95 initialActions Sum of mean exceeds: 380ms | Sum of p95 exceeds: 518.8ms Sum of all benchmark exceeds: 898.8ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
adonesky1
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
When a dapp connects to MetaMask via EVM and the user rejects a subsequent connection request for SVM, the existing EVM connection is closed. This can potentially break dapp flows and edge cases where maintaining the EVM connection is critical.
To fix this, we implemented partial revoking into the
wallet_revokeSessionhandler on MetaMask/core#6668 so that we can then update our solana wallet-standard provider would only revoke the solana scopes when it is asked to disconnect, here MetaMask/solana-wallet-standard#53.This PR bumps all necessary packages for this to take place, fixing the undesired EVM permissions revoke when a solana wallet standard connection is refused.
Changelog
CHANGELOG entry: Fixes existing EVM permissions removed when a Solana Wallet Standard connection is rejected
Related issues
Fixes: https://consensyssoftware.atlassian.net/browse/WAPI-413
Manual testing steps
window.ethereum.enable()eth_accountsshould still be present with the permitted account incaveatsarray.Screenshots/Recordings
Before
Screen.Recording.2025-09-18.at.6.18.40.PM.mov
After
Screen.Recording.2025-09-23.at.15.45.34.mov
Pre-merge author checklist
Pre-merge reviewer checklist