VirusTotal_Download (v.1.1.2) service failing with errors

[ricotj]

New install of crits (master) with the VirusTotal_Download service (master).

I get the following error when trying to download a sample:
Failed to create new Sample: handle_file() got an unexpected keyword argument 'reference'

Taking a look at the crits/crits/samples/handlers.py
def handle_file seems to have changed between stable and current master,
reference is now source_reference
method is now source_method

with these fixed (VirusTotal_Download init.py line# 186 & 188) I am getting the following error:
Failed to create new Sample: 'str' object has no attribute 'has_access_to'

[brlogan]

That new error is probably a user/username issue. The user object has a "has_access_to" attribute, but if only the username is passed (not the entire user object), then you'll get that error.

[phate1]

for anyone else that hits this I resolved by modifying the init.py file in the virustotal_download_service folder right near the end when it sends the sample to crits to match this:

202             try:
203                 handle_file(filename = obj.md5,
204                             data = data,
205                             source = "VirusTotal",
206                             source_reference = "Binary downloaded from VT based on MD5",
207                             user = self.current_task.user,
208                             source_method = "VirusTotal Download Service",
209                             md5_digest = obj.md5 )

[apolkosnik]

Nice, can you create a pull request? Thanks!

[phate1]

sure #350