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 requirements.txt #1

Closed
wants to merge 1 commit into from
Closed

Add requirements.txt #1

wants to merge 1 commit into from

Conversation

pirate
Copy link

@pirate pirate commented Mar 1, 2016

Not sure if it's complete since I still got several errors even with all the required packages installed, but it's a start.

Ideally there should also be a setup.py to install python-dev and the crypto libraries that need to be installed via apt-get. Would be nice to have some setup instructions or a quickstart in the readme too.

@codemunkee
Copy link

I was going to send along the same thing, also probably worth adding dnet, pycrypto, and pcapy.

@PierrePaul
Copy link

Worked for me, but I'm getting these errors :

Import Error - most likely due to missing pycrypto libraries - disabling crypto functionality
ImportError('cannot import name TLSHandshake',)

Not really sure of the impact, might be related to circular import and not related at all to this requirements.txt

@pirate
Copy link
Author

pirate commented Mar 1, 2016

@PierrePaul those are the same errors I got.

@pdf
Copy link

pdf commented Mar 1, 2016

See tintinweb/scapy-ssl_tls@951faa4 for the circular import fix

@blha303 blha303 mentioned this pull request Mar 1, 2016
@martinseener
Copy link

On OSX El Capitan i was able to use scanner.py by installing the following using Homebrew:
brew install python (installs latest 2.7 including pip)
pip install enum pycrypto scapy scapy-ssl_tls pyasn1

Afterwards i still get the Import Error like @PierrePaul above but the script actually checks my server then. It's maybe worth adding pyasn1 to the requirements as well, @pirate

nimia pushed a commit that referenced this pull request Mar 1, 2016
add dependency information and fix up readup
@nimia
Copy link
Owner

nimia commented Mar 1, 2016

Thanks for helping make this tool more accessible, your input is greatly appreciated.
It looks like the updated README.me mentions all relevant dependencies.
I'm aware of this message:
ImportError('cannot import name TLSHandshake',)
that confuses people, and am working on removing it.
Closing this issue for now, please let me know if there are more missing dependencies, or anything that prevents people from using the tool. Thanks!

@nimia nimia closed this Mar 1, 2016
@AxisNL AxisNL mentioned this pull request Mar 5, 2016
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.

None yet

6 participants