-
Notifications
You must be signed in to change notification settings - Fork 71
Virus Total integration #84
Comments
The packages do this on submission to the community repo. There may be benefit to doing it ahead of time as well. |
Yeah, I know. However, the benefit is substantial:
au_BeforeUpdate() {
Get-RemoteFiles
$vt = Get-VirusTotalPositives
if ( $vt -gt 5 ) {
Write-Host "Ignoring package due to virus total results - $vt positives
return 'ignore'
}
} |
There is definitely a benefit of doing this ahead of time, like for instance when a package provides multiple downloads (like the thunderbird and firefox packages, for different cultures). Since the community repo only catches the default, users of other languages and using VirusTotal will get a question to upload it to VirusTotal (usually happens with me all the time), which also makes choco lose track of the package itself (I believe a bug for that had been submitted, but not sure) As well as the two points that @majkinetor mentioned of course. @majkinetor regarding your second point, not completely sure if that would be possible within a reasonable time frame. |
Yeah.. kinda invalidates the entire idea for fast repositories such as core team repo, unless
|
This would be fairly easy to do, embedded packages could check on VT and ignore on too many posistives
The text was updated successfully, but these errors were encountered: