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

OAuth2 client list service is maybe ambiguous #579

Open
margaretha opened this issue Apr 3, 2023 · 3 comments
Open

OAuth2 client list service is maybe ambiguous #579

margaretha opened this issue Apr 3, 2023 · 3 comments

Comments

@margaretha
Copy link
Contributor

The parameter authorized_only in the OAuth2 Client list service is possibly ambiguous.

Only OAuth2 clients registered by a user is listed when authorized_only=false (default).

When authorized_only=true, Kustvakt does not really filter the user-registered clients, but lists all authorized clients, including those not registered by the user himself.

Kustvakt should probably include all authorized clients when authorized_only=false. We need to show which clients owned/have been registered by the users.

Maybe registered_by should be removed from the response for data security because it would show usernames of other users.

@hebasta
Copy link

hebasta commented Apr 3, 2023

registered_by:
I agree that it is a data security sensible subject. On the other side it might be for some users a helpful information, for example, if you want to install only clients of a certain institution, etc.
All in all we probably should delete it, but keep that in mind.

@margaretha
Copy link
Contributor Author

registered_by has been omitted as also discussed in #572.

@margaretha
Copy link
Contributor Author

We agreed to keep the existing behavior of the web-service, and deprecate the parameter authorized_only. It should then be removed from the system in the next versions.

Another more appropriate parameter filter_by should be introduced with two possible string value authorized_only or owned_only. The parameter is optional, when it is not set, it should list all clients. But as long as authorized_only exists, it has a higher priority, so the web-service will behave according to the value of authorized_only.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants