-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
feat: enable support to Blockaid on custom networks on mobile #8292
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. |
👍 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. Ignoring: Next stepsTake a deeper look at the dependencyTake a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support [AT] socket [DOT] dev. Remove the packageIf you happen to install a dependency that Socket reports as Known Malware you should immediately remove it and select a different dependency. For other alert types, you may may wish to investigate alternative packages or consider if there are other ways to mitigate the specific risk posed by the dependency. Mark a package as acceptable riskTo ignore an alert, reply with a comment starting with |
bc3d8d4
to
820c673
Compare
@SocketSecurity ignore npm/react-native-quick-crypto@0.6.1 |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #8292 +/- ##
==========================================
+ Coverage 40.34% 40.37% +0.02%
==========================================
Files 1235 1235
Lines 29957 29951 -6
Branches 2877 2876 -1
==========================================
+ Hits 12087 12092 +5
+ Misses 17175 17165 -10
+ Partials 695 694 -1 ☔ View full report in Codecov by Sentry. |
## **Description** Fix blockaid mobile multichain. Re-init PPOM instance on network switch. ## **Related issues** Fixes: MetaMask/MetaMask-planning#1926 ## **Manual testing steps** 1. Open app and enable blockaid 2. Switch to mainnet, wait for few seconds and go to test dapp to submit malicious transaction - validation should work 3. Switch to any other network supported by blockaid and wait for few seconds and go to test dapp to submit malicious transaction - validation should work ## **Screenshots/Recordings** NA ## **Pre-merge author checklist** - [X] I’ve followed [MetaMask Coding Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md). - [X] I've clearly explained what problem this PR is solving and how it is solved. - [X] I've linked related issues - [X] I've included manual testing steps - [ ] I've included screenshots/recordings if applicable - [ ] I’ve included tests if applicable - [ ] I’ve documented my code using [JSDoc](https://jsdoc.app/) format if applicable - [X] I’ve applied the right labels on the PR (see [labeling guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)). Not required for external contributors. - [X] I’ve properly set the pull request status: - [X] In case it's not yet "ready for review", I've set it to "draft". - [ ] In case it's "ready for review", I've changed it from "draft" to "non-draft". ## **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. --------- Co-authored-by: Sylva Elendu <iamsylvaelendu@gmail.com>
…twork coverage on mobile (#8327) ## **Description** This is a follow up to https://github.com/MetaMask/MetaMask-planning/issues/1693. Once that PR is merged, we should adjust copy to communicate the support for custom networks. What's new copy should be updated to: Steer clear of known scams while still preserving your privacy with security alerts powered by Blockaid. This feature is available on Arbitrum, Avalanche, BNB chain, Ethereum Mainnet, Linea, Optimism and Polygon. Always do your own due diligence before approving requests. Settings copy should be updated to: Privacy preserving - no data is shared with third parties. Available on Arbitrum, Avalanche, BNB chain, Ethereum Mainnet, Optimism and Polygon. <!-- Write a short description of the changes included in this pull request, also include relevant motivation and context. Have in mind the following questions: 1. What is the reason for the change? 2. What is the improvement/solution? --> ## **Related issues** Fixes: ## **Manual testing steps** 1. Go to this page... 2. 3. ## **Screenshots/Recordings** <!-- If applicable, add screenshots and/or recordings to visualize the before and after of your change. --> ### **Before** <!-- [screenshots/recordings] --> ### **After** <!-- [screenshots/recordings] --> ## **Pre-merge author checklist** - [ ] I’ve followed [MetaMask Coding Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md). - [ ] I've clearly explained what problem this PR is solving and how it is solved. - [ ] I've linked related issues - [ ] I've included manual testing steps - [ ] I've included screenshots/recordings if applicable - [ ] I’ve included tests if applicable - [ ] I’ve documented my code using [JSDoc](https://jsdoc.app/) format if applicable - [ ] I’ve applied the right labels on the PR (see [labeling guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)). Not required for external contributors. - [ ] I’ve properly set the pull request status: - [ ] In case it's not yet "ready for review", I've set it to "draft". - [ ] In case it's "ready for review", I've changed it from "draft" to "non-draft". ## **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.
12fa06d
to
38c2e4c
Compare
E2E test started on Bitrise: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/5e98ad0e-d7b1-4191-83fd-2c1792232630 |
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
Description
This is a follow up to #7547. Once this PR is merged, we'll be able to support additional networks on extension and we should enable the feature on them.
We should enabled it on the following networks:
Ethereum Mainnet (already done)
BNB chain
Polygon
Arbitrum
Optimism
Avalanche
Linea
Related issues
Fixes: https://github.com/MetaMask/MetaMask-planning/issues/1693
Manual testing steps
Screenshots/Recordings
Before
After
http://recordit.co/jC95wVGXEi
Screen.Recording.2024-01-18.at.15.46.27.mov
Pre-merge author checklist
Pre-merge reviewer checklist