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

CryptographyDeprecationWarning with a seemingly simple fix #105

Open
vahp opened this issue May 28, 2024 · 1 comment
Open

CryptographyDeprecationWarning with a seemingly simple fix #105

vahp opened this issue May 28, 2024 · 1 comment

Comments

@vahp
Copy link

vahp commented May 28, 2024

getting this straightforward-looking warning when running vapid

$ vapid --gen
/Users/me/myproject/venv/lib/python3.11/site-packages/py_vapid/init.py:200: CryptographyDeprecationWarning: Curve argument must be an instance of an EllipticCurve class. Did you pass a class by mistake? This will be an exception in a future version of cryptography.
self.private_key = ec.generate_private_key(ec.SECP256R1, default_backend())

it seems to me that you can fix it with with a pair of parens : ec.SECP256R1()
I'm leery of messing with other people's security code so just opening this as an issue.

@jrconlin
Copy link
Member

Sigh.

The fun of dependent library changes...

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

2 participants