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

Client name and client identifier are differents #10

Open
weierophinney opened this issue Dec 31, 2019 · 2 comments
Open

Client name and client identifier are differents #10

weierophinney opened this issue Dec 31, 2019 · 2 comments

Comments

@weierophinney
Copy link
Contributor

In (ClientEntity, ClientRepository, ClientTrait, entityTrait) The code treats clientIdentifier === clientName,

OR , in "League\OAuth2\Server\Entities\ClientEntityInterface" , clientIdentifier and clientName are two different fields.

why not to seperate them as ligue-oauth2 do ?

what is the interest? can you answer me please?,
Maybe you have things to be planned.


Originally posted by @samiboukadida at zendframework/zend-expressive-authentication-oauth2#7

@Danielss89
Copy link
Contributor

Regarding the client identifier, there is a bit of a mixup. The client identifier in the league oauth package is a string name, and this is persisted in this mezzio package as client_name. But ths mezzio package also has a client_id, which is where the mixup happens. In theory, the client_id should be a varchar 100(in all the tables), and the repository should be changed to find the client from the client_id and not client_name(which should not be the unique identifier here).
I can do a PR with this change, if the solution is agreeable by the maintainers(I'm not sure who this is, @Ocramius @weierophinney ?).

@Danielss89
Copy link
Contributor

@froschdesign I just saw you merging another commit, so pinging you here too. See above comment :)

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