-
Notifications
You must be signed in to change notification settings - Fork 11
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
Malware protection 6: Malware integration #3604
Changes from 63 commits
afd0746
45c982d
f041a1a
3cdf77d
aca479c
712f7d6
21f4585
d135c21
5687819
35cf020
8cb5739
6f29625
87fd08c
9ef094d
be5bde1
bafd937
fcc6627
eb8be0b
906ccd4
59b989a
4d40826
87f5f47
fc9db2d
e380729
5b82f9b
30460c2
94f1595
961e60f
556ec25
caccaff
5445910
df70113
46dce01
a5efd99
200c1b2
78640c2
5c53151
6a688ac
0eac1a9
83338a8
6b92133
df4639b
c9c7a46
5449841
5afbe3b
31a6325
8b4c30e
82553b8
b14f1a3
4d2ac61
7efd315
6062a7f
aeee5df
4bbae0a
89bfa8b
fee4525
18656dc
7fd0faf
b5ea63e
8b62077
ae17370
e465120
61c94a0
88dbfb1
be2f2ee
941332a
3ee8985
8893893
d672847
c92cb45
4564fcf
01c9932
f39d20f
3b80c27
6307071
dfb58cb
d53d4f6
28f752e
890480a
d4c60ee
244467d
e14814f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -31,8 +31,8 @@ | |
switch (dataType, dataType.threatKind) { | ||
case (.hashPrefixSet, .phishing): "phishingHashPrefixes.json" | ||
case (.filterSet, .phishing): "phishingFilterSet.json" | ||
// case (.hashPrefixes, .malware): "malwareHashPrefixes.json" | ||
// case (.filters, .malware): "malwareFilterSet.json" | ||
case (.hashPrefixSet, .malware): "malwareHashPrefixes.json" | ||
case (.filterSet, .malware): "malwareFilterSet.json" | ||
} | ||
} | ||
|
||
|
@@ -46,11 +46,12 @@ | |
struct EmbeddedDataProvider: MaliciousSiteProtection.EmbeddedDataProviding { | ||
|
||
private enum Constants { | ||
static let embeddedDataRevision = 1694418 | ||
static let phishingEmbeddedHashPrefixDataSHA = "d9eccd24d05ce16d4ab877574df728f69be6c7840aea00e1be11aeafffb0b1dc" | ||
// TODO: Rollback the revision and filter (with -f) set when malware is available on the server | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can we remove it? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yes, this is to be removed before merging. for now it‘s there to test |
||
static let embeddedDataRevision = 1695638 | ||
static let phishingEmbeddedHashPrefixDataSHA = "a9fdca1e3c852ac288b738314732847c0daf27caf3ed30a349ecaaae04682e7e" | ||
static let phishingEmbeddedFilterSetDataSHA = "5452a5a36651c3edb5f87716042175b5a3074acb5cc62a279dbca75479fc1eda" | ||
// static let malwareEmbeddedHashPrefixDataSHA = "be5a2320307ed0dd8b8b2f2702dbade752dfb886aae24f212b0c3009524636aa" | ||
// static let malwareEmbeddedFilterSetDataSHA = "37517e5f3dc66819f61f5a7bb8ace1921282415f10551d2defa5c3eb0985b570" | ||
static let malwareEmbeddedHashPrefixDataSHA = "b248a417fbd003de95fd273ae01d1b336fe2599b7d1c60b8f71786a943ab4a83" | ||
static let malwareEmbeddedFilterSetDataSHA = "6c63a19c15868083f80b9936cafc482ae9e6f01d27246aa91789197b6f5ba9e7" | ||
} | ||
|
||
func revision(for dataType: MaliciousSiteProtection.DataManager.StoredDataType) -> Int { | ||
|
@@ -69,8 +70,8 @@ | |
switch (dataType, dataType.threatKind) { | ||
case (.hashPrefixSet, .phishing): Constants.phishingEmbeddedHashPrefixDataSHA | ||
case (.filterSet, .phishing): Constants.phishingEmbeddedFilterSetDataSHA | ||
// case (.hashPrefixes, .malware): Constants.malwareEmbeddedHashPrefixDataSHA | ||
// case (.filters, .malware): Constants.malwareEmbeddedFilterSetDataSHA | ||
case (.hashPrefixSet, .malware): Constants.malwareEmbeddedHashPrefixDataSHA | ||
case (.filterSet, .malware): Constants.malwareEmbeddedFilterSetDataSHA | ||
} | ||
} | ||
|
||
|
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.
we need translations
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.
yes, as soon as we get our copy approved