Remove dependencies on Cryptodome and oscrypto
The code to use cryptography (which uses OpenSSL) already existed, but it just wasn't being used by default. Since cryptography is currently a mandatory dependency, we may as well use it all the time.
Partially resolves #1605/SNOW-843716.
Please answer these questions before submitting your pull requests. Thanks!
-
What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
Fixes #NNNN
-
Fill out the following pre-review checklist:
- [ ] I am adding a new automated test(s) to verify correctness of my new code
- [ ] I am adding new logging messages
- [ ] I am adding a new telemetry message
- [ ] I am modifying authorization mechanisms
- [ ] I am adding new credentials
- [ ] I am modifying OCSP code
- [ ] I am adding a new dependency
-
Please describe how your code solves the related issue.
Please write a short description of how your code change solves the related issue.
CLA Assistant Lite bot:
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.
I have read the CLA Document and I hereby sign the CLA
2 out of 3 committers have signed the CLA.
:white_check_mark: (sfc-gh-sfan)[https://github.com/sfc-gh-sfan]
:white_check_mark: (sfc-gh-jdu)[https://github.com/sfc-gh-jdu]
:x: @geofft
You can retrigger this bot by commenting recheck in this Pull Request
@geofft I think the linters are still failing, and requires CLA?