-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Modernize Virustotal API scraping #327
base: master
Are you sure you want to change the base?
Conversation
- Replace old Virustotal URL with new one - Add required Virustotal headers required for successful request
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.
Did almost the same thing for VirusTotal without using API key!
👍🏻
It worked. Tnx! |
works fine, thanks for the fix. |
I'm still learning to Linux here and I'm in douobt which file to change... When doing a search i find it at 2 locations Thanks :) |
theHarvester is another tool, you need to edit /usr/lib/python3/dist-packages/sublist3r.py |
The fix works, merge it! |
Can we get this merged please? cc: @aboul3la |
# Virustotal requires specific headers to bypass the bot detection: | ||
self.headers["X-Tool"] = "vt-ui-main" | ||
self.headers["X-VT-Anti-Abuse-Header"] = "hm" # as of 1/20/2022, the content of this header doesn't matter, just its presence | ||
self.headers["Accept-Ianguage"] = self.headers["Accept-Language"] # this header being present is required to prevent a captcha |
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.
@AetherBreeze self.headers["Accept-Ianguage"] should this not be corrected to self.headers["Accept-Language] ? My guess is this might cause an undefined variable error.
it worked , thanks |
I am new to linux and using sublist3r. please it possible to provide a step by step instruction on how I can solve the problem. Because I tried using the API key and is not working. thanks |
Please How did you do it? |
use this repo. |
thanks Man. it is working. |
Can we merge this change or is this project now considered being in "development hell"? If it works, WHY can't we commit this? Hello? EDIT: Looking at @aboul3la 's front page it seems they're either a: working full-time b: dead (hopefully not) or c: this project has been abandoned. I'm going with the latter being c: Project maintenance has been abandoned. This is a great tool, sad to see even PR's are not being merged. If anyone is looking for a working version that respects the new changes use @AetherBreeze 's forked repo here: |
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.
working as good
Error: Virustotal probably now is blocking our requests |
maybe it is possibility to decompile the inofficial VirusTotal Mobile app to have again a rate limit free scraping method without api key. i haven't gotten that far yet, but it seems like the particular "partner" is calling urls to scan |
works fine for me, thanks brother! |
Almost 4 years late, sorry:
|
Address #194 by modernizing the Virustotal scraper.
Some time in the past few years, Virustotal added a few verification methods to the subdomain API. This pull request changes the endpoint and adds headers so that the request succeeds.
This solves the same problem as #285, but does not require an API key and uses the existing parser logic.
This pull request has been tested on Python 3.9.2, Kali Linux rolling 2021/2.