Closed byjl closed 3 years ago
This is probably due to a problem with the JWT class and has been documented in both Laravel passport and other projects. I released a version 2.0.0 of my package a few days ago which should work, but it also requires Passport 10.1.0 or higher, which made changes to also fix the same issues with the JWT package. There's probably some mix of versions in between where things remain unstable.
Here you can read more about it: https://github.com/laravel/passport/issues/1381
Basically the JWT package made some changes that causes hard errors in all kinds of projects including laravel passport. I don't see any fixes in the Passport 9 version tree, so I guess passport isnt being fixed for 9.0. This means really the only option is upgrading to 10.1.0+ (and my package 2.0.0), or lock the jwt package on 3.3.3 like people have been doing according to that issue thread.
Thank you for pointing me to that thread. I will override the package class until I am able to upgrade my project to use Passport 10 and laravel-passport-claims 2.
& for anyone who may be reading this, locking lcobucci/jwt to 3.3.3 caused issues with authenticating via cookies somehow
Thanks
I see that passport 9 also made some changes to accomodate this problem, but in the PHP8 commit, that's why I didnt see it. I'll see if I can get this solved in the 1.x version (passport 9) as well.
I just pushed version 1.8.0, which depends on Passport 9.4.0+. I think that works, at least if I install Laravel7 + Passport9.4 + my package version 1.8.0 I can create tokens with custom claims. Let me know when you have the chance. You probably need to remove your 3.3.3 lock, else it wont install Passport 9.4 I think.
yup, that worked, thanks a lot!
awesome, thanks for testing
I recently did a composer update on my project which resulted in an issue with laravel-passport-claims. This is the exception I was getting when creating an access token:
I fixed it by overriding this class: CorBosman\Passport\AccessToken @ convertToJWT with the following changes:
Thanks