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

user_id is not passed in ClientCredentials #47

Open
RuslanZavacky opened this issue Dec 23, 2016 · 1 comment
Open

user_id is not passed in ClientCredentials #47

RuslanZavacky opened this issue Dec 23, 2016 · 1 comment

Comments

@RuslanZavacky
Copy link

From what I see in GrantType/ClientCredentials.php in method getUserId we request $this->clientData['user_id'].
Going to Storage/ClientCredentials.php it seems this storage do not return user_id. Also, in oauth_client table there is no column user_id.

Then, if we have ability to create custom UserProvider, it means that this provider should be used when retrieving user.

Question is - it is intentional, or it might be implemented or what is the appropriate solution in this case?

@bshaffer
Copy link
Owner

bshaffer commented Jan 9, 2017

@RuslanZavacky Good question. This is marked as optional in ClientInterface. So I think the short answer is this library doesn't support it, and that should be ok (as it's not part of the oauth spec).

However, getClientDetails should return a null value for user_id to prevent a PHP warning.

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

No branches or pull requests

2 participants