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

Fixed bug related to UTF-8 in certain requests #93

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

P3GLEG
Copy link

@P3GLEG P3GLEG commented Feb 20, 2017

Fixed bug in domain_censys.py not printing the given ip as well

@P3GLEG
Copy link
Author

P3GLEG commented Mar 12, 2017

I added Dockerimages for general use and development as well. Love your tool!

Copy link

@Chan9390 Chan9390 left a comment

Choose a reason for hiding this comment

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

There is already a docker image (https://hub.docker.com/r/appsecco/datasploit/) available for Datasploit maintained by @madhuakula. The only difference is that his Dockerfile uses Ubuntu 16.04 docker image whereas you have used Debian Jessie

&& apt-get remove -y libxml2-dev python-dev libxmlsec1-dev curl wget build-essential \
&& rm -rf /var/lib/apt/lists/*

USER k4ch0w

Choose a reason for hiding this comment

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

Why is the user k4ch0w ?

Copy link
Author

Choose a reason for hiding this comment

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

The user was left over from deving. My bad, I didn't see the dockerimage was already made. :( I will remove my image then and add his link to the documentation for new users of the project.

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