search

joselfonseca / lighthouse-graphql-passport-auth

Add GraphQL mutations to get tokens from passport for https://lighthouse-php.com/
https://lighthouse-php-auth.com/
MIT License
228 stars 56 forks source link

Fix RefreshToken mutation by limiting lcobucci/jwt to ^4.0 #166

Closed wimski closed 1 year ago

wimski commented 1 year ago

  1. laravel/passport:^11.0 requires lcobucci/jwt:^4.3|^5.0 https://github.com/laravel/passport/releases/tag/v11.8.5

  2. lcobucci/jwt:^5.0 removed Lcobucci\JWT\Configuration::forUnsecuredSigner() https://lcobucci-jwt.readthedocs.io/en/latest/upgrading/#removal-of-none-algorithm

This broke the RefreshToken mutation, so I've limited the dependency for now as a quick fix. Ideally the method call should be replaced, but I don't have the knowledge or time to do this.