Skip to content
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

Fix for VirusTotal error #285

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix for VirusTotal error #285

wants to merge 2 commits into from

Conversation

crahan
Copy link

@crahan crahan commented Nov 21, 2020

This PR fixes the outdated VirusTotal API endpoint which has been reported in #194. It uses the value specified in the VT_APIKEYenvironment variable. If the the API key is missing, it prints an error message and skips the VirusTotal engine.

Screen Shot 2020-11-22 at 00 38 26

@GiangTieuDau
Copy link

I did fix accordingly to yours
And I also changed VT_APIKEY into my API key
But the error keep showing
[!] Error: VirusTotal API key environment variable not found. Skipping

How to fix that?

@crahan
Copy link
Author

crahan commented Nov 23, 2020

@GiangTieuDau try the following on lines 690 and 691.

self.headers.update({'X-ApiKey': self.apikey})
print(self.headers)
resp = self.session.get(url, headers=self.headers, timeout=self.timeout)
print(resp.json())

And check the value of the headers you're sending and the response JSON that gets returned. Either the API key is not being sent, the API is not correct and causes an error, or some other error is happening. Printing either of those values should get you more details.

@RoninNakomoto
Copy link

I did fix accordingly to yours
And I also changed VT_APIKEY into my API key
But the error keep showing
[!] Error: VirusTotal API key environment variable not found. Skipping

How to fix that?
@GiangTieuDau @crahan
export the API KEY And run the tool without the sudo. It will solve ur problem.
you are running the tool as sudo while you're exporting the API_KEY into the current user's environment.

@Mayor294
Copy link

[!] Error: Virustotal probably now is blocking our requests

😑😑😑😑😑.... Someone help me out Thank You

Copy link

@irjohn irjohn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This fix worked for me. Thanks for the patch brother

@ashifly
Copy link

ashifly commented Jun 29, 2022

Worked for me. Thank you :))

@owenrrr
Copy link

owenrrr commented Aug 25, 2022

This actually helps me thx!

Copy link

@SalemNabeelSalem SalemNabeelSalem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's working with me and fixing the issue of Error: Virustotal probably now is blocking our request

@SkySlic3
Copy link

@GiangTieuDau try the following on lines 690 and 691.

self.headers.update({'X-ApiKey': self.apikey})
print(self.headers)
resp = self.session.get(url, headers=self.headers, timeout=self.timeout)
print(resp.json())

And check the value of the headers you're sending and the response JSON that gets returned. Either the API key is not being sent, the API is not correct and causes an error, or some other error is happening. Printing either of those values should get you more details.

Where do I type this into

@soukainazay
Copy link

it's not working for me i've tried all the things discussed, can any one help me out please :(

@soukainazay
Copy link

@GiangTieuDau try the following on lines 690 and 691.

self.headers.update({'X-ApiKey': self.apikey})
print(self.headers)
resp = self.session.get(url, headers=self.headers, timeout=self.timeout)
print(resp.json())

And check the value of the headers you're sending and the response JSON that gets returned. Either the API key is not being sent, the API is not correct and causes an error, or some other error is happening. Printing either of those values should get you more details.

Where do I type this into

in the Sublist3r.py file (type gedit ./sublist3r.py to modify) i've tried it and it's not working for me. did u find another way to solve this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants