Skip to content
This repository was archived by the owner on May 12, 2026. It is now read-only.

Control access to all libraries by default when enableAuthorization = false#258

Open
Ryllaz wants to merge 4 commits into
9p4:mainfrom
Ryllaz:deny-libraries-access-for-new-users-by-default-if-authorization-disabled
Open

Control access to all libraries by default when enableAuthorization = false#258
Ryllaz wants to merge 4 commits into
9p4:mainfrom
Ryllaz:deny-libraries-access-for-new-users-by-default-if-authorization-disabled

Conversation

@Ryllaz

@Ryllaz Ryllaz commented Apr 16, 2025

Copy link
Copy Markdown

This PR resolves the issue #147.
It will set EnableAllFolders setting to the value from Enable All Folders checkbox of plugin's settings.
That allows admin to prevent full library access for new users.

So if the plugin does not control authorization and the checkbox Enable All Folders is set to false - new user will be created without full library access, and that is more secure.

By default EnableAllFolders = true. It will override the setting according to setting in SSO plugin Enable All Folders

@Ryllaz Ryllaz changed the title Deny all libraries access by default when enableAuthorization = false Control access to all libraries by default when enableAuthorization = false Apr 16, 2025
@9p4

9p4 commented Apr 18, 2025

Copy link
Copy Markdown
Owner

Won't this will cause adjustments to the users' access to be overwritten during the next login?

@9p4 9p4 added question Further information is requested stale Waiting for response for a while labels May 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

question Further information is requested stale Waiting for response for a while

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants