-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
system install (as a module) support and setup.py for easier installation #52
Comments
I noted that there is a new sqlite database in use. The DB should be generated in homedir ~/.theHarverster |
@blshkv would you want to create an install.sh for this? There is currently a dockerfile and you can also git clone it. Sorry for the late response! |
no, that would be a wrong approach. It has to be setup.py which should list the required dependencies and define system location of all files. The tool should also support system-wide installation, i.e. installed under root into |
@blshkv would you want to work on this and submit a pull request? |
Sorry, I don't have bandwidth for development of other tools. Just a simple bug reporting (like this issue) and maintaining hundreds of packages takes a fair amount of time which I could afford. |
Hi All, I have updated the documentation for the dependencies but it needs to be merge and when you run the tool for the first time it creates the sqlite DB |
@blshkv My changes have been merged can you take a look at the README.md please? |
Not sure what to look for. Does the tool support system install using setup.py? Will it only from the current folder? Yes. |
The dependencies section and will look to getting the Kali patch upstreamed which should fix this
…________________________________
From: Anton Bolshakov <notifications@github.com>
Sent: Tuesday, January 1, 2019 5:36:08 AM
To: laramies/theHarvester
Cc: J.Townsend; Comment
Subject: Re: [laramies/theHarvester] system install support and setup.py for easier installation (#52)
Not sure what to look for. Does the tool support system install using setup.py?
No.
Will it only from the current folder? Yes.
So nothing changed
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub<#52 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ADddQpAcTC_n3OLlZ54aELf-iCgHHfmOks5u-vPIgaJpZM4MmG8z>.
|
I have tried it, still long way to go. setup.py is a python file, which usually tells you that the module/package you are about to install has been packaged and distributed with Distutils, which is the standard for distributing Python Modules. This allows you to easily install Python packages. Often it's enough to write:
and the module will install itself. https://docs.python.org/3/installing/index.html#installing-index |
@blshkv what do you think of the PR for setup.py? |
I have tried to convert the harvester into a proper python module but this is a major re-design and I'm unable to test all functions. As I said, I have no time to rewrite it, however I will be happy to test your progress. I can also help with minor fixes, once the direction of the development is clear. |
Please review the new setup.py file that was added. Would like to close out this ticket. |
Tested and failed. Have you tested at all?
You need to convert it into python module, redirect all outputs into user's dir etc . Still long way to go |
Regarding the wordlists, please use the following example: |
@blshkv please try now I tested it after the most recent commit and it seems to be working :) |
worldlist files are installed into an unusual location: |
FYI, you can debug it with the following command: |
#229 |
@blshkv if you are satifised with the changes that have happened feel free to close :) |
@NotoriousRebel thank you for accepting the patch. That would be the first stage, I can install the tool as a system package now (using root account). The second stage is to redirect all output to a userdir, (something like https://github.com/vulnersCom/getsploit/pull/14/files which I mentioned before) and redirect all output to ~/.theHarvester so a regular user would be able to run it without touching system files |
@blshkv Would you agree that this issue could be closed now? As I think we now for fill the subject of this thread and open a new thread to do with the other bits and pieces that you have mentioned. |
Ok, sure. Thanks for accepting my patch once again |
Ported certspotter from requests to aiohttp + misc. fixes
There are few minor changes would be required. Something like this:
The text was updated successfully, but these errors were encountered: