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 pycrypto to requirements.txt #288

Merged
merged 2 commits into from Oct 18, 2016
Merged

Conversation

SamJoan
Copy link
Contributor

@SamJoan SamJoan commented Oct 16, 2016

Hi there,

In my case, when following the instructions on the cowrie installation page, I got this error:

(cowrie-env)cowrie@hostname:~/cowrie$ ./start.sh cowrie-env
Activating virtualenv "cowrie-env"
Starting cowrie with extra arguments [] ...
An error has occurred: b'ImportError: No module named Crypto.PublicKey'
Please look at log file for more information.

I am not sure why this happens, and nobody else has opened an issue for this bug, but the solution is simple so I thought I'd make a PR.

Cheers,
Pedro

@ryd994 ryd994 mentioned this pull request Oct 17, 2016
@ryd994
Copy link

ryd994 commented Oct 17, 2016

Do you want to remove crypto ?
As it is actually an gpg encrypt package.
Check https://pypi.python.org/pypi/crypto

@SamJoan
Copy link
Contributor Author

SamJoan commented Oct 17, 2016

That seems like it'd make sense, I've removed it from my branch.

@micheloosterhof micheloosterhof merged commit 6aeeb72 into cowrie:master Oct 18, 2016
@micheloosterhof
Copy link
Member

Thanks!

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

3 participants