refactor: Remove openssl feature flag

[cdmurph32]

Changes in this pull request

Moving cryptography to c2pa-crypto made the openssl and openssl_sign flags defunct.

Checklist

• This PR represents a single feature, fix, or change.
• All applicable changes have been documented.
• Any TO DO items (or similar) have been entered as GitHub issues and the link to that issue has been included in a comment.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.76%. Comparing base (91f24a9) to head (ee4ecb8).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #940   +/-   ##
=======================================
  Coverage   78.75%   78.76%           
=======================================
  Files         147      147           
  Lines       34965    34963    -2     
=======================================
- Hits        27538    27537    -1     
+ Misses       7427     7426    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[gpeacock]

Thanks, this looks like a very nice code clean up.
I'm curious about how an application will be able to not build with openssl now.
The openssl_sign feature was added for c2pa_min which shows how to build c2pa for signing only using no SSL.

[cdmurph32]

Thanks, this looks like a very nice code clean up. I'm curious about how an application will be able to not build with openssl now. The openssl_sign feature was added for c2pa_min which shows how to build c2pa for signing only using no SSL.

I think they would add c2pa-crypto as an explicit dependency with the feature rust_native_crypto. I think that has been true since @scouten-adobe moved the crypto to its own crate.