Budibase / budibase

Low code platform for building business apps and workflows in minutes. Supports PostgreSQL, MySQL, MariaDB, MSSQL, MongoDB, Rest API, Docker, K8s, and more 🚀
https://budibase.com
Other
21.78k stars 1.49k forks source link

[BUDI-7623] "Manage Roles" on the User table is not useful #3412

Closed mjashanks closed 11 months ago

mjashanks commented 2 years ago

The Users table has the option to Manage Roles, just like any other table.

This works. When I set Read = Power and try to access as Basic, I am prevented.

However, it seems like every app makes an initial call to /api/row/ta_users - presumably so it can be used for enriching users bindings? This causes an error on in the app, if Read access is restricted on the user table.

So, currently, it makes no sense to be able to restrict read access to the user table.

However, I do think that it makes sense for every logged in user to have a list of all the other users in the app. It would also be nice to restrict access to the user table, to stop everyone from seeing other user meta-data (i.e. custom columns).

I suggest that we have 2 separate endpoints:

Originally from discussion: https://github.com/Budibase/budibase/discussions/3368#discussioncomment-1658137

BUDI-7623

stale[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity.

stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity.