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

Laravel 11 #176

Closed wsslfnstr closed 3 months ago

wsslfnstr commented 3 months ago

I have combined the 2 existing pr's to hopfully fix the tests

wsslfnstr commented 3 months ago

Update, tests will fail. I will look into it tomorrow

SlyDave commented 3 months ago

I believe the tests are failing because of the inclusive of JWT @ version 5.0 as noted here https://github.com/joselfonseca/lighthouse-graphql-passport-auth/issues/167 and here https://github.com/joselfonseca/lighthouse-graphql-passport-auth/pull/166

SlyDave commented 3 months ago

Laravel Passport 12 still supports version 4. "lcobucci/jwt": "^4.3|^5.0", So the dirty fix is to keep lighhouse-graphql-passport-auth at version ^4.3 for now, until a proper fix can be applied.