Extension of the SessionInterface about oauth2-server HOT 5 CLOSED

Interesting - we've also implemented the same functionality but it can be done without editing the library's code.

We've done it by having an extra column in the scopes database that limits a scope to "autonomous only" clients (i.e. where the user doesn't have to be involved) and then in class that implements the SessionInterface I also return this flag along with the other required parameters. Likewise the class that implements ClientInterface returns an additional flag if a client is autonomous only. Then I just do some additional checks in my own code using existing functions.

I don't think at the moment it is worth implementing in the library, but I will write a tutorial to show how to do it.

from oauth2-server.

I generally know what you mean, but how do you get the required information "using existing functions"?

If you use the Client Credentials Grant you can get the client id from the parameters, okay, but if you use the Password Grant, the user login (where you get the user id) is done directly before the scope check and there is no chance to get it - so I would not only have to check all parameters again in my scope class, but also do the login and other checks a second time.

This could be avoided by the extension - and the existing functionality would not be influenced as the additional parameters don't have to be used.

from oauth2-server.

Am just going to mention #21 in here as both requests are asking for similar requirements (i.e. limiting clients to certain scopes/grants)

from oauth2-server.

I concur with @ziege. Those usecases are quite common with OAuth, and updates described in #20 & #21 seem to me more good practice than hacking.

It's not about cluttering the library with a bunch of methods rarely used, just to think about possible advanced checks and extend parameters/return values in interfaces accordingly.

It does not complexify their implementation, and API requiring advanced permission controls could be developped without forcing the programmers to extend half the library classes.

As this library is one of the cleanest and most up-to-date implementation of OAuth 2 in PHP, it really should not miss an opportunity to support (not implement) out-of-the-box advanced usecases.

from oauth2-server.

Now implemented in the dev branch. See #24 comments.

Going to close this issue.

from oauth2-server.