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

Add /usr/local/etc as possible location for config files #550

Merged
merged 1 commit into from
Oct 6, 2020

Conversation

alexmerkel
Copy link
Contributor

The change make theHarvester search for proxies.yaml etc. in /usr/local/etc/theHarvester/ directory in addition to /etc/theHarvester and the current directory. This allows system-wide configs to be installed without root privileges, e.g. via Homebrew.

Additionally, I refactored the code in core.py at bit. Now there is one method to read the api-keys.yaml file and all service-specific methods just call this function and pass on the right key from the returned object.

@L1ghtn1ng
Copy link
Collaborator

@alexmerkel Thanks for this and this is call that you have added homebrew support, the other thing that you will need to look at though is the stash.py for the sqlite DB or does the current way we do it work? it would need to stay that way but like how you have done the api-keys if it needs to be modified

@alexmerkel
Copy link
Contributor Author

The location of the sqlite database is fine imo. It is written to a user controlled location (~/.local), so installing (and running) theHarvester as a non-root user works. I added the other file locations because the original ones are in a location (/etc) typically not writable for normal users.

@L1ghtn1ng L1ghtn1ng merged commit b90cfbc into laramies:master Oct 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants