forked from laravel/passport
-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pullrequests/laravel/10.x #2
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
# Conflicts: # CHANGELOG.md
[8.x] Shouldn't the migrations use bigIncrements and indexes on relationships?
Signed-off-by: Kevin Ullyott <[email protected]>
Signed-off-by: Kevin Ullyott <[email protected]>
[8.x] Add a Passport Client factory to Passport publishing
…-password-grant [8.x] Add tests for getting the access token with the password grant
[8.x] Revoke refresh tokens as well
[8.x] Implement auth token for access requests
[8.x] Forget session keys on invalid match
* Laravel 9 support * Update composer.json * Update tests.yml
* Allow to use custom authorization server response * Update src/Passport.php Remove direct initialisation as "null" Co-authored-by: Dries Vints <[email protected]> * Update Passport.php Co-authored-by: Dries Vints <[email protected]> Co-authored-by: Taylor Otwell <[email protected]>
* Update UPGRADE.md I just went through the steps of upgrading passport from v8 to v10 and hit a minor snag that could have been solved by a tweak to the documentation. When upgrading from v8 to v9 it says place two Passport static calls "within the `boot` method of your `AppServiceProvider`" When upgrading from v9 to v10 it says you can remove the calls from your `AuthServiceProvider`. I don't claim to know where those calls belonged in the first place, so this proposal just updates the documentation to list them both. * Update UPGRADE.md Co-authored-by: Taylor Otwell <[email protected]>
When using this factory in feature tests with `$this->withoutDeprecationHandling();` call the test fails as the deprecation gets converted to an exception: ``` Since fakerphp/faker 1.14: Accessing property "company" is deprecated, use "company()" instead. ```
The firebase/php-jwt recently had to make breaking changes to resolve a security flaw in their library. This change upgrades that library and fixes the code that broke with the upgrade.
# Conflicts: # composer.json # config/passport.php # database/factories/ClientFactory.php # database/migrations/2016_06_01_000001_create_oauth_auth_codes_table.php # database/migrations/2016_06_01_000002_create_oauth_access_tokens_table.php # phpunit.xml.dist # src/Bridge/Client.php # src/Bridge/ClientRepository.php # src/Bridge/UserRepository.php # src/Client.php # src/ClientRepository.php # src/Console/ClientCommand.php # src/Console/HashCommand.php # src/Console/InstallCommand.php # src/Console/PurgeCommand.php # src/Guards/TokenGuard.php # src/Http/Controllers/AuthorizedAccessTokenController.php # src/Http/Controllers/ClientController.php # src/Http/Controllers/PersonalAccessTokenController.php # src/Http/Controllers/RetrievesAuthRequestFromSession.php # src/Http/Middleware/CheckClientCredentials.php # src/Http/Middleware/CheckClientCredentialsForAnyScope.php # src/Http/Middleware/CheckCredentials.php # src/Http/Rules/RedirectRule.php # src/Passport.php # src/PassportServiceProvider.php # src/PassportUserProvider.php # src/PersonalAccessTokenFactory.php # src/RefreshTokenRepository.php # src/Token.php # src/TokenRepository.php # tests/Unit/ApproveAuthorizationControllerTest.php # tests/Unit/AuthorizedAccessTokenControllerTest.php # tests/Unit/BridgeAccessTokenRepositoryTest.php # tests/Unit/BridgeClientRepositoryTest.php # tests/Unit/CreateFreshApiTokenTest.php # tests/Unit/DenyAuthorizationControllerTest.php # tests/Unit/PersonalAccessTokenControllerTest.php # tests/Unit/TokenGuardTest.php
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.