Skip to content

feat: dapp scanning enabled on transactions and signatures #33829

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

imblue-dabadee
Copy link
Contributor

@imblue-dabadee imblue-dabadee commented Jun 23, 2025

Description

Currently, we only have dapp scanning enabled when the user connects to a dapp. However, there is a possibility the user may not ever see the trust signal for the URL in a confirmation window (see here for an elaborated explanation). Having it enabled here reduces that possibility.

Open in GitHub Codespaces

Pre-merge author checklist

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.

Copy link
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.

@metamaskbot metamaskbot added team-product-safety Push issues to Product Safety team INVALID-PR-TEMPLATE PR's body doesn't match template labels Jun 23, 2025
@imblue-dabadee imblue-dabadee enabled auto-merge June 23, 2025 09:39
@metamaskbot
Copy link
Collaborator

Builds ready [d163cc0]
UI Startup Metrics (1218 ± 64 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1218110815046412481345
load105395313436010771167
domContentLoaded104794913396010711160
domInteractive18144161734
firstPaint68584134942610601125
backgroundConnect7423279
firstReactRender23165572339
getState1263271728
initialActions001001
loadScripts79870595753819909
setupStore85213818
WebpackHomeuiStartup22921736298824024262695
load17431295227120518482128
domContentLoaded17371290225220318412123
domInteractive181289141457
firstPaint1766443069199318
backgroundConnect2410248242645
firstReactRender24790426118376411
getState1243461526
initialActions316135
loadScripts17331288224020118382111
setupStore237326432136
FirefoxBrowserifyHomeuiStartup13761229169811014321633
load1210107414138412771365
domContentLoaded1209107314128412771365
domInteractive1163734657119313
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2413141182251
firstReactRender24215642431
getState11513318930
initialActions001001
loadScripts1189106113988312631343
setupStore74314714
WebpackHomeuiStartup15871380212415716841965
load13691194180613114721601
domContentLoaded13691193180613114721601
domInteractive88302954491144
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21164952335
firstReactRender41325444448
getState136111151131
initialActions002111
loadScripts13501173178513114551575
setupStore95557820
Benchmark value 18 exceeds gate value 17 for chrome browserify home p95 setupStore
Benchmark value 2292 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 1744 exceeds gate value 1711 for chrome webpack home mean load
Benchmark value 1737 exceeds gate value 1704 for chrome webpack home mean domContentLoaded
Benchmark value 248 exceeds gate value 235 for chrome webpack home mean firstReactRender
Benchmark value 1734 exceeds gate value 1699 for chrome webpack home mean loadScripts
Benchmark value 2696 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 2128 exceeds gate value 2030 for chrome webpack home p95 load
Benchmark value 2124 exceeds gate value 2005 for chrome webpack home p95 domContentLoaded
Benchmark value 58 exceeds gate value 57 for chrome webpack home p95 domInteractive
Benchmark value 411 exceeds gate value 370 for chrome webpack home p95 firstReactRender
Benchmark value 2112 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 116 exceeds gate value 110 for firefox browserify home mean domInteractive
Benchmark value 12 exceeds gate value 11 for firefox browserify home mean getState
Benchmark value 313 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 30 exceeds gate value 24 for firefox browserify home p95 getState
Benchmark value 42 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 1965 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Sum of mean exceeds: 225ms | Sum of p95 exceeds: 798ms
Sum of all benchmark exceeds: 1023ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 196 Bytes (0%)
  • ui: 0 Bytes (0%)
  • common: 0 Bytes (0%)

Copy link
Contributor

@AugmentedMode AugmentedMode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
INVALID-PR-TEMPLATE PR's body doesn't match template team-product-safety Push issues to Product Safety team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants