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

Import ABC from collections.abc instead of collections for Python 3 compatibility #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tirkarthi
Copy link

Fixes #10

@SpudInNZ
Copy link

SpudInNZ commented Dec 9, 2021

This is now causing python-jose-cryptodome to not work with Python 3.10 - can somebody with write access please merge and release this PR?

@thomaskosciuch
Copy link

@markhu - would it be possible to merge?

@satu
Copy link

satu commented Nov 5, 2022

will this ever be merged and release? I bumped into this while using https://github.com/capless/warrant library.

@markhu
Copy link

markhu commented Nov 5, 2022

This is not my repo, so I can't merge.

@Yamen-Malik
Copy link

Yamen-Malik commented Jul 30, 2023

Did anyone find a work around or an alternative?
I've fixed the problem manually for development but this won't work on production (too janky).

UPDATE:
I ended up using pyJWT module.

If anyone is going to do the same make sure to convert the RSA key from JWK (dictionary) to PEM format. To do that use the following line:
jwt.algorithms.RSAAlgorithm.from_jwk(json.dumps(rsa_key))

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.

Importing ABC directly from collections was deprecated from 3.4 and will be removed in 3.10
6 participants