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

Remove use of cryptography's default_backend() #248

Merged
merged 1 commit into from
Feb 25, 2025

Conversation

MasterKale
Copy link
Collaborator

I stumbled on to the fact that the backend arg has been silently ignored since cryptography 3.1:

https://cryptography.io/en/latest/faq/#what-happened-to-the-backend-argument

cryptography stopped requiring the use of backend arguments in version 3.1 and deprecated their use in version 36.0. If you are on an older version that requires these arguments please view the appropriate documentation version or upgrade to the latest release.

Note that for forward compatibility backend is still silently accepted by functions that previously required it, but it is ignored and no longer documented.

I decided to remove all use of default_backend() that was being passed in for the backend arg to take advantage of this fact. I think it makes for more readable code.

@MasterKale MasterKale merged commit ff0f34d into master Feb 25, 2025
5 checks passed
@MasterKale MasterKale deleted the fix/delete-vestigial-default-backend branch February 25, 2025 19:20
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

Successfully merging this pull request may close these issues.

1 participant