django-graphql-auth icon indicating copy to clipboard operation
django-graphql-auth copied to clipboard

Update dependencies: PyJWT and django-graphql-jwt

Open lucmult opened this issue 3 years ago • 8 comments

Currently other JWT libraries that used with django-graphql require PyJWT > 2.0, e.g: django-graphql-jwt

Remove PyJWT from the dependencies from django-graphsql-auth is safe because this isn't imported directly here.

Upgraded django-graphql-jwt from 0.3.0 to 0.3.2.

I've run the following tests locally:

$ make test
...
  py38-django30: commands succeeded
  congratulations :)

Lint and format run locally too.

lucmult avatar Jul 05 '21 12:07 lucmult

This should fix #111

lucmult avatar Jul 05 '21 12:07 lucmult

This is great.

amorino avatar Jul 22 '21 00:07 amorino

@PedroBern can you merge this PR?

cadiente-jomel avatar Jan 13 '22 08:01 cadiente-jomel

Did you actually get it to install? I cloned it with your modifications and did:

cd django-graphql-auth
pip install -e .

result:

ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
gql 2.0.0 requires graphql-core<3,>=2.3.2, but you have graphql-core 3.1.7 which is incompatible.
Successfully installed PyJWT-2.3.0 aniso8601-9.0.1 django-graphql-auth-0.3.16 django-graphql-jwt-0.3.2 graphene-3.0 graphene_django-3.0.0b7 graphql-core-3.1.7 graphql-relay-3.1.5

pors avatar Feb 03 '22 08:02 pors

Please, can we get this trivial PR merged? :pray:

PabloCastellano avatar Mar 15 '22 17:03 PabloCastellano

Please, can we get this trivial PR merged? 🙏

@PabloCastellano I think it is not that trivial, see my comment above. These dependencies exist for a reason.

pors avatar Mar 15 '22 17:03 pors

@pors That error is not related to this PR. If you clone master and do the same, you'll get incompatibilities too:

ERROR: graphql-relay 3.1.5 has requirement graphql-core<3.2,>=3.1, but you'll have graphql-core 2.3.2 which is incompatible.
ERROR: graphene 3.0 has requirement graphql-core~=3.1.2, but you'll have graphql-core 2.3.2 which is incompatible.
ERROR: graphene-django 2.15.0 has requirement graphene<3,>=2.1.7, but you'll have graphene 3.0 which is incompatible.

PabloCastellano avatar Mar 16 '22 11:03 PabloCastellano

@pors check out my pr https://github.com/PedroBern/django-graphql-auth/pull/147 which supersedes this one

PabloCastellano avatar Mar 16 '22 12:03 PabloCastellano