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.

Sign up for GitHub

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

Jump to bottom

Import Errors #40

Closed
nimia opened this issue Mar 3, 2016 · 0 comments · Fixed by #43
Closed

Import Errors #40

nimia opened this issue Mar 3, 2016 · 0 comments · Fixed by #43

Comments

@nimia
Copy link
Owner

nimia commented Mar 3, 2016

The tool emits the following import errors under some circumstances:

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

As the README file states, these errors are admittedly annoying, but do not prevent the tool from producing correct results.

At this point, I am unable to chase down the source of this problem, as it does not reproduce for me.
Kind users who have encountered this error and can suggest a fix, are encouraged to (preferably) submit a pull request, or add a comment to this issue.

Thanks in advance,
Nimrod
@nimia nimia mentioned this issue Mar 3, 2016
Closed
@tintinweb tintinweb mentioned this issue Mar 3, 2016
Merged
@nimia nimia closed this as completed in #43 Mar 4, 2016
nimia added a commit that referenced this issue Mar 4, 2016
@nimia
Merge pull request #43 from tintinweb/fix/imports
6d33bf7 
Removes sys.path magic.
Renames scapy-ssl_tls to scapy_ssl_tls and adds __init__.py in order allow direct imports. (dashes are not allowed in package names for imports).
Removes unused import for ssl_tls_crypto.
Removes circular import in scapy_ssl_tls.ssl_tls trying to import ssl_tls_crypto which in return tried to import from ssl_tls.
Fixes issue #40.
Thank you so much tintinweb.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix/imports #40 tintinweb/public_drown_scanner
1 participant
@nimia