-
Notifications
You must be signed in to change notification settings - Fork 5.4k
fix: dynamic parameters via sig_params #37086
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
|
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. |
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [1f7b1ef]
UI Startup Metrics (1194 ± 93 ms)
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 [73add40]
UI Startup Metrics (1247 ± 82 ms)
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 [306e213]
UI Startup Metrics (1257 ± 67 ms)
Bundle size diffs [🚀 Bundle size reduced!]
|
|
❌ 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 |
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [5eeae0b]
UI Startup Metrics (1271 ± 86 ms)
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 [1c9ed70]
UI Startup Metrics (1291 ± 85 ms)
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 [f889979]
UI Startup Metrics (1253 ± 86 ms)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Description
Adds support for sig_params in deeplinks
Changelog
CHANGELOG entry: null
Related issues
Fixes: https://github.com/MetaMask/MetaMask-planning/issues/6017
Manual testing steps
Screenshots/Recordings
Not applicable
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Introduce
sig_paramssupport in deep link canonicalization, parsing, signing, and tests to allow selectively signed/forwarded query params.canonicalizeto honorsig_params— keep only listed query params (preserving repeats), appendsig_params, sort, and removesig; fallback to previous behavior whensig_paramsis absent.sig_paramsbefore invoking route handlers, and verify against the original URL.SIG_PARAMS_PARAM(sig_params).sig_paramsfiltering, repetition preservation, non-mutation; adjustparsetests to pass empty params to handlers when only signature present.sig_paramsand unsigned; add assertions that only allowed params flow through; addgetHashParamshelper; update redirects to ignore query in expectations.signDeepLinkto optionally appendsig_params(derived from current params) and sign the full URL; add imports and sorting adjustments.Written by Cursor Bugbot for commit f889979. This will update automatically on new commits. Configure here.