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

Error Message on Launch - Mac OSX Yosemite #23

Closed
ForceLightning opened this issue Jul 1, 2015 · 8 comments
Closed

Error Message on Launch - Mac OSX Yosemite #23

ForceLightning opened this issue Jul 1, 2015 · 8 comments

Comments

@ForceLightning
Copy link

Traceback (most recent call last):
File "/Users/totallynotme/Downloads/theHarvester-master/theHarvester.py", line 10, in
from discovery import *
File "/Users/totallynotme/Downloads/theHarvester-master/discovery/googlesearch.py", line 6, in
import requests
ImportError: No module named requests

@darker0n
Copy link

darker0n commented Jul 4, 2015

sudo pip install discovery
sudo pip install requests

@kaizensoze
Copy link
Contributor

I think pip install requests should suffice. I actually overlooked the Dependencies section myself in the README since it's right next to the Changelog so it might make sense to include a pip install requests.

@laramies
Copy link
Owner

laramies commented Jul 5, 2015

Yes, the only dependency is requests, I added a check to see if requests is installed, and if not will tell the user to install the library. Also I added in the README the dependency. Will upload later today.

@laramies
Copy link
Owner

laramies commented Jul 5, 2015

Btw, latest version (2.6) has both things. Can't you see them?

@kaizensoze
Copy link
Contributor

After doing a pip uninstall requests and running the latest version I see the same thing that @ForceLightning228 is reporting.

@laramies
Copy link
Owner

laramies commented Jul 5, 2015

But requests should be installed, you need to have it in order to run theHarvester, if you do an uninstall (pip uninstall requests) it is normal that it wont work. Can you confirm that it works when requests is installed? thanks.

@kaizensoze
Copy link
Contributor

Sorry maybe I misunderstood your comment. I thought you were saying you added a check to see if requests is installed and, if not, give a custom message instead of a Traceback.

@laramies
Copy link
Owner

laramies commented Jul 5, 2015

You are right, that is how it is supposed to work. But I know what is the problem. I am importing first Discovery, which will import the discovery modules and those are using requests; so it fails there before the check i added in the main file. I will fix this, thanks for raising it.

L1ghtn1ng pushed a commit that referenced this issue Feb 15, 2020
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

No branches or pull requests

4 participants