Skip to content
This repository has been archived by the owner on Dec 6, 2024. It is now read-only.

Remove cryptography requirement #32

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

Remove cryptography requirement #32

wants to merge 1 commit into from

Conversation

com4
Copy link
Contributor

@com4 com4 commented Jan 4, 2021

  • django-oauth-toolkit-jwt does not directly rely on the cryptography
    library so it is not needed. Pinning the old version has the potential
    to create dependency issues.

@com4
Remove cryptography requirement
2a88b8c 
- django-oauth-toolkit-jwt does not directly rely on the cryptography
  library so it is not needed. Pinning the old version has the potential
  to create dependency issues.
rafa-munoz
rafa-munoz reviewed Feb 23, 2021
View reviewed changes
Copy link

@rafa-munoz rafa-munoz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For any reason, Travis is not building anymore. Have you checked that it's running the tests successfully?

@Brunux
Copy link

Brunux commented Jul 15, 2021

Facing this issue actually

django-oauth-toolkit-jwt 0.7.0 requires cryptography~=2.8, but you have cryptography 3.4.7 which is incompatible.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@rafa-munoz rafa-munoz rafa-munoz left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@com4 @Brunux @rafa-munoz