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

openssl_pkcs12: add cryptography backend #21

Closed
felixfontein opened this issue Apr 5, 2020 · 4 comments · Fixed by #234
Closed

openssl_pkcs12: add cryptography backend #21

felixfontein opened this issue Apr 5, 2020 · 4 comments · Fixed by #234
Labels
enhancement New feature or request

Comments

@felixfontein
Copy link
Contributor

SUMMARY

The openssl_pkcs12 module should support a cryptography backend, so PyOpenSSL can be deprecated / removed eventually. Currently cryptography does not support PKCS12, though, so some preliminary work on adding PKCS12 support to cryptography is needed. Also, this means that deprecating the PyOpenSSL backend would need to be done much later than for other modules, since a new enough cryptography supporting the new features (which aren't there yet) will take some years to trickle through.

(Migration of ansible/ansible#59905)

ISSUE TYPE
  • Feature Idea
COMPONENT NAME

openssl_pkcs12

@felixfontein
Copy link
Contributor Author

cryptography has now basic PKCS12 suport: https://cryptography.io/en/latest/hazmat/primitives/asymmetric/serialization/#pkcs12

It seems to be too limited for what the openssl_pkcs12 module offers, though, so it's not usable for us yet.

@MarkusTeufelberger
Copy link
Contributor

Looks like it was added in pyca/cryptography#5325 to cryptography. Let's hope they expand on it.

@felixfontein felixfontein added the enhancement New feature or request label Mar 14, 2021
@felixfontein
Copy link
Contributor Author

What's currently missing to replicate the current behavior is the ability to read the 'friendly name' from the PKCS#12 file. I've created pyca/cryptography#5760 for that.

@felixfontein
Copy link
Contributor Author

resolved_by #234.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants