-
Notifications
You must be signed in to change notification settings - Fork 654
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
Added Windows support. #104
base: staging
Are you sure you want to change the base?
Conversation
How extensively has this been tested? |
Hi Tim, it has been properly tested both on Windows and Linux, debugging to make sure everything it's working fine. I've also installed modules from the marketplace, loading and running them. Moreover, I've integrated Elasticsearch with Recon-ng (recon-es), developing several modules (recon-es-marketplace). Here you have the technical paper, it's my final project for the Software Engineering degree: TFG Regards. |
Interesting. So it just replaces SQL with ES for setting and retrieving the default module sources? The rest of the code base looks unchanged. |
Basically, yes. An index is created in Elasticsearch for each Recon-ng workspace, where documents containing the collected data are indexed, organised by type of data. In addition, due to the lack of duplicity control in Elasticsearch as the autogenerated _id is independent of the indexed data, the _id used is instead the hashed dictionary values to avoid data redundancy. |
The parsing of paths seems a bit complicated (using regexp). I suggest using os.sep to split paths into components. Example: |
b8e879d
to
9e907df
Compare
Is this already merged? |
No description provided.