Skip to content

Enable crypto library flexibility #54

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

Closed
bifurcation opened this issue Apr 21, 2025 · 1 comment
Closed

Enable crypto library flexibility #54

bifurcation opened this issue Apr 21, 2025 · 1 comment

Comments

@bifurcation
Copy link
Contributor

Right now, things are hard-wired to OpenSSL -- OpenSSL 1.1, even. It would be nice to upgrade to the OpenSSL 3 series, or maybe support mBedTLS, WolfSSL, etc. I expect the way to do this is probably roughly as libsrtp has, with uniform interface and a .cpp file per back-end (see, e.g., all the aes_gcm_${LIB}.c files in the crypto/cipher diretory). Then you can switch between them via the build system.

It may be that the existing crypto.cpp is isolated enough that it's the right target. At a minimum, you would want to do things like rename openssl_error to crypto_library_error and make sure there were no OpenSSL dependencies outside of crypto.cpp.

@bifurcation
Copy link
Contributor Author

Fixed in #58

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

1 participant