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

sflvault requires the unmaintained pycrypto library #55

Open
Apteryks opened this issue Feb 10, 2021 · 3 comments
Open

sflvault requires the unmaintained pycrypto library #55

Apteryks opened this issue Feb 10, 2021 · 3 comments

Comments

@Apteryks
Copy link

There's at least one CVE which has gone unfixed, https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2018-6594. The corresponding (unfixed) issue is pycrypto/pycrypto#253.

There's an actively maintained forked of pycrypto which is https://github.com/Legrandin/pycryptodome, but it removed rather than fixed the support for ElGamal encryption/decryption that sflvault relies on, so migration is non trivial.

Also see https://www.chenweikeng.com/elgamal.html and the question here: Legrandin/pycryptodome#504.

@Apteryks
Copy link
Author

Apteryks commented Feb 10, 2021

The CVE linked above seems to only affects applications not using hybrid (public/private) crypto, which sflvault does, so it should be safe, FWIW.

@maximeconnolly
Copy link

There is actually some hybrid crypto going on in the background deep somewhere if my memories serve because back when I used to work at SFL, we actually did talk about that CVE more than once.

There is no way to go forward since pycryptodome has no support for ElGamal.

@Apteryks
Copy link
Author

Apteryks commented Feb 18, 2021

@maximest-pierre Thanks for tipping in. The author of pycryptodome points to RSA as a suitable replacement for ElGamal; we should evaluate how complex a transition to it would be.

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

2 participants