Skip to content

Conversation

khast3x
Copy link
Contributor

@khast3x khast3x commented Apr 14, 2019

Hey there,

I hope this works for you. Copy pasting the details from the first PR below:

Best,


Updated the Dockerfile to be much simpler, and to use a stable Metasploit+Postgresql Docker image as base instead of the previous awkward install.

Since its now pre-configured, the Dockerfile is compatible for auto-building on Dockerhub at every push (it's free). I attached an example in the documentation using the build I did creating this fork. If you decide to use that feature and set it up, it'll be up to date every time you push 😎

You'll just need to create an account, link github, and create a build for AutoSploit. Since the Dockerfile is in the Docker/ folder, be sure to set it in the autobuild configuration like so.

Tried it on an EC2. The Datacenter speed+AutoSploit combo was pretty ridiculous 🚀

Finally, if you do decide to have an auto-build setup, I'll be sure to add AutoSploit to my side project. Trying out the AutoSploit container to Multi-Tor container, results are very nice.

Let me know if you have any questions/feedbacks before merging.

Regards,

@Ekultek
Copy link
Contributor

Ekultek commented Apr 14, 2019

Looks good to me @NullArray?

@NullArray
Copy link
Owner

Hey guys, thank you for the pull request @khast3x it's always appreciated bro. I'll go over the commit real quick and add my review. After which i will merge it.

@@ -1,75 +1,19 @@
# Docker deployment instructions
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should probably reflect the changes in the main README.md as well. Or at least leave a reference to the readme that currently lives in the docker directory.

Wiki entry might be appropriate as well.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright i did the main README.md for now. Just wanted to post that here.

Copy link
Owner

@NullArray NullArray left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As i wrote in the review comments, the updated Docker readme should probably be reflected in the main README.md as well and i think a wiki entry would be cool as well.

As an aside...
I'll write the new official release when i have some more time tomorrow, if it hasn't been done already. I will include @Ekultek 's new feature and the improvement on the operations related to docker.

As always i appreciate you guys' contribution <3

@NullArray NullArray merged commit 6976580 into NullArray:dev-beta Apr 16, 2019
@khast3x
Copy link
Contributor Author

khast3x commented Apr 16, 2019

You're welcome, was great seeing it roll smoothly with the clean MSF+DB base.
Let me know if you make an auto-build, would be really neat!

Best,

@NullArray
Copy link
Owner

You're welcome, was great seeing it roll smoothly with the clean MSF+DB base.
Let me know if you make an auto-build, would be really neat!

Best,

Yeah man, for sure. If we make an auto-build you'll be the first to know 👍

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

Successfully merging this pull request may close these issues.

3 participants