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

Migrate away from pycrypto? #6

Closed
smithfarm opened this issue Dec 6, 2018 · 4 comments
Closed

Migrate away from pycrypto? #6

smithfarm opened this issue Dec 6, 2018 · 4 comments
Assignees

Comments

@smithfarm
Copy link
Contributor

Hi. I noticed that ceph-iscsi has a runtime dependency on pycrypto 2.6.1 [1]

This is problematic for openSUSE because this package has been removed from the distribution. It is slated to be dropped from SLES as well, though that has not happened just yet.

Apparently, this package is only used here:

from Crypto.PublicKey import RSA

The openSUSE maintainers suggest that we migrate to https://pypi.org/project/pycryptodomex/ or https://pypi.org/project/pyOpenSSL/ - is this feasible?

[1] https://www.dlitz.net/software/pycrypto/

@dillaman
Copy link

dillaman commented Dec 6, 2018

I'd vote for pyOpenSSL since Ceph already has a dependency on it.

@rjfd
Copy link
Contributor

rjfd commented Dec 6, 2018

@dillaman @smithfarm the pyOpenSSL only works for managing keys/certificates it does not have the encrypt/decrypt function that it's required by the ceph-iscsi code.

In the the pyOpenSSL documentation they recommend the usage of python-cryptography package to preform encrypt/decrypt (https://pyopenssl.org/en/stable/api/crypto.html#module-OpenSSL.crypto).

Can we use python-cryptography instead?

@smithfarm
Copy link
Contributor Author

@rjfd Works for me:

SLE-15: python3-cryptography 2.1.4
openSUSE:Factory: python3-cryptography 2.3.1

@dillaman
Copy link

dillaman commented Dec 6, 2018

That's fine w/ me as well since it's available in RHEL/CentOS

@rjfd rjfd assigned rjfd and unassigned dillaman and ricardoasmarques Dec 6, 2018
rjfd added a commit to rjfd/ceph-iscsi that referenced this issue Dec 6, 2018
Fixes: ceph#6

Signed-off-by: Ricardo Dias <rdias@suse.com>
rjfd added a commit to rjfd/ceph-iscsi that referenced this issue Dec 7, 2018
Fixes: ceph#6

Signed-off-by: Ricardo Dias <rdias@suse.com>
rjfd added a commit to rjfd/ceph-iscsi that referenced this issue Dec 7, 2018
Fixes: ceph#6

Signed-off-by: Ricardo Dias <rdias@suse.com>
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

No branches or pull requests

4 participants