-
Notifications
You must be signed in to change notification settings - Fork 4.8k
[12.x] Passport 13.x #9819
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
base: 12.x
Are you sure you want to change the base?
[12.x] Passport 13.x #9819
Conversation
*/ | ||
public function boot(): void | ||
{ | ||
Passport::authorizationView('auth.oauth.authorize'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We probably need to instruct users what this view should actually contain and what it should do on approval / denial.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instruction has been added.
public function boot(): void | ||
{ | ||
Passport::deviceUserCodeView('auth.oauth.device.user-code'); | ||
Passport::deviceAuthorizationView('auth.oauth.device.authorize'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here, we should tell people what to put in these views.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instruction has been added.
Passport 13.x
EnsureClientIsResourceOwner
middleware passport#1794CheckClientCredentials
middleware passport#1792OAuthServerException
passport#1763oauth_scopes
property of the bearer token onTokenGuard
passport#1755