passport
passport copied to clipboard
[13.x] Deprecate JSON API
trafficstars
This PR deprecates JSON API because:
- Not needed anymore as we now have better option: Jetstream OAuth feature, UI included and fully customizable #1771
- It adds 10
webroutes, 4 controllers, mostly not reusable, not customizable, and without frontend; Makes Passport feature bloated without adding much value. - The are a few bugs in the current implementation (I can list them all if you want!), didn't get that much update during years (shows the fact that most people don't use it?), too opinionated and just makes Passport hard to maintain.
Deprecations
\Laravel\Passport\Http\Controllers\AuthorizedAccessTokenControllercontroller class.\Laravel\Passport\Http\Controllers\ClientControllercontroller class.\Laravel\Passport\Http\Controllers\PersonalAccessTokenControllercontroller class.\Laravel\Passport\Http\Controllers\ScopeControllercontroller class\Laravel\Passport\RefreshTokenRepositoryrepository class.\Laravel\Passport\TokenRepositoryrepository class.\Laravel\Passport\Http\Rules\RedirectRulerule class.\Laravel\Passport\Http\Rules\UriRulerule class.\Laravel\Passport\ClientRepository::findForUser,forUser,activeForUser,revoked,update, anddeleterepository methods.
Changes
- Adds
Passport::$registersJsonApiRoutes = false. - Doesn't register JSON API routes by default anymore.
- Makes 2 unit test classes feature tests.
Upgrade Guide
You may call Passport::$registersJsonApiRoutes = true if you want to continue using deprecated JSON API.