Skip to content
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

Patch 1 #1234

Closed
wants to merge 2 commits into from
Closed

Patch 1 #1234

wants to merge 2 commits into from

Conversation

tomjamon
Copy link

@tomjamon tomjamon commented May 4, 2020

Last december a change was made to allow any valid client :

Based on the theory and official standards of OAuth2: "The Client Credentials grant is used when applications request an access token to access their own resources, not on behalf of a user." (REF1, REF2).

Shouldn't this change be persistant ?

(taylorotwell merged commit on 5 Dec 2019)

Ref
#1125
#1132

Last december a change was made to allow any valid client :

Based on the theory and official standards of OAuth2: "The Client Credentials grant is used when applications request an access token to access their own resources, not on behalf of a user." (REF1, REF2).

Shouldn't this change be persistant ?

(taylorotwell merged commit on 5 Dec 2019)

Ref
laravel#1125
laravel#1132
@driesvints
Copy link
Member

This has been fixed already on master. Passport 9 will be released tomorrow.

@driesvints driesvints closed this May 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants