-
-
Notifications
You must be signed in to change notification settings - Fork 184
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: Block Client Side C2 Requests by Managing a hashed C2 Request Blocklist #4526
Merged
Merged
Changes from all commits
Commits
Show all changes
59 commits
Select commit
Hold shift + click to select a range
9006ace
wip: implemented logic to support managing a c2 request blocklist / c…
df06c98
wip: succesfully blocks c2 requests - need to fix the blocklist inges…
dbb8981
wip: succesfully blocks c2 requests - need to fix the blocklist inges…
2cd59e0
feat: Add ethereum-cryptography dependency for phishing-controller pa…
b874d37
fix: utils shah256 now uses hostname - Phishing controller now sucesf…
AugmentedMode 8b50223
100% test coverage woo
AugmentedMode cb2b899
fix prettier issue
AugmentedMode 75f7151
Update yarn.lock
AugmentedMode 28ec91c
merge with main to solve merge conflicts
AugmentedMode 018b836
chore: Update CLIENT_SIDE_DETECION_BASE_URL and REQUEST_BLOCKLIST_END…
AugmentedMode 07e1c14
refactor: Update phishing lists with recently added and removed entries
AugmentedMode 4e159a8
chore: Update HOTLIST_REFRESH_INTERVAL to 5 minutes
AugmentedMode 73c8fc8
fix: pr comments
AugmentedMode 139138a
small cleanup of tests
AugmentedMode af256a1
fix pr comments
AugmentedMode 2b61d3b
fix tests
AugmentedMode cdf48b1
chore: update UserStorageController type (#4551)
Prithpal-Sooriya 2e95453
feat: notification-services-controller - add shared notification bloc…
Prithpal-Sooriya 33bd597
phishing-controller owned by product-safety (#4554)
409H 1a9d68f
feat: add `getNFTContractInfo` (#4524)
OGPoyraz 41ff05b
chore: Remove redundant dependencies (#4556)
Gudahtt 103822b
feat: fixing snap connect issues (#4560)
dovydas55 e9aa1e9
Release 180.0.0 (#4548)
MajorLift a5559c5
fix: Compress transaction history upon update (#4555)
Gudahtt 59b5eb8
Release/181.0.0 (#4562)
Gudahtt a68ade0
Convert constraints from Prolog to JavaScript (#4546)
mcmire eb0ba1d
remove provider undefined type (#4567)
kanthesha 75f8867
fix: throw error when removing selected network (#4566)
mikesposito 232a73b
feat: add unlock checks for notification and profile sync controllers…
Prithpal-Sooriya d298f83
feat: add `KeyringController:addNewAccount` messenger action (#4565)
mathieuartu d9f41c3
feat: add new method `updateAccountMetadata` (#4568)
mathieuartu 587e3e8
Fix/default network (#4557)
kanthesha 38da8f5
feat: update notifications controllers from extension fixes. (#4573)
Prithpal-Sooriya f6e81a0
Add public `name` property to `AssetsContractController` class (#4564)
MajorLift dfdaab2
Merge branch 'main' into feat/client-side-detection
AugmentedMode b2a92b4
fix yarn
AugmentedMode 2329ba4
Merge branch 'main' into feat/client-side-detection
AugmentedMode e46b791
Merge branch 'main' into feat/client-side-detection
AugmentedMode 68af08a
updated the c2 blocklist base url
AugmentedMode fcf5a9b
Merge branch 'feat/client-side-detection' of https://github.com/MetaM…
AugmentedMode c40f834
chore: Refactor PhishingController.ts to simplify conditional stateme…
AugmentedMode 33266c5
chore: fixed merge conflicts
AugmentedMode 5365452
chore: fix yarn lock from main
AugmentedMode c652eab
chore: Handle URL parsing errors in PhishingDetector.ts
AugmentedMode a36a332
chore: fix tests to cover invalid url
AugmentedMode dfe8a8a
chore: fix yarn lock
AugmentedMode f9e412c
chore: refactor PhishingDetectorResult type to be an enum PhishingDet…
AugmentedMode 378cd0e
chore: fix tests with new type PhishingDetectorResultType
AugmentedMode 42fa317
chore: fix tests with new type PhishingDetectorResultType
AugmentedMode 973b47c
chore: fix try catch with url hostname in check function
AugmentedMode d8ba223
chore: Add roundToNearestMinute function for rounding Unix timestamps
AugmentedMode 7c35d95
refactor: updateC2DomainBlocklist now has its own function to handle …
AugmentedMode f31b98a
chore: revert pulling time of c2 blocklist back to 15 minutes
AugmentedMode 3b4ca95
chore: export PhishingDetectorResult type from phishing controller to…
AugmentedMode 7c0e1dc
Merge branch 'main' into feat/client-side-detection
AugmentedMode 6a95e69
chore: cleanup phishing controller tests
AugmentedMode 14bb4e7
Merge branch 'feat/client-side-detection' of https://github.com/MetaM…
AugmentedMode 7c3ba9d
chore: clean up tests
AugmentedMode a7a5420
chore: update comments for documentation
AugmentedMode File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
The addition of these dependencies should be noted as
CHANGED
entries in the changelog, possibly as breaking changes.