-
Notifications
You must be signed in to change notification settings - Fork 17
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
Allow enabling only for specified groups #20
Comments
Something like this would require a new table to be created. |
I would like to emphasize the ability to restrict user login for specific users. That way, I could have my admin account only be accessible from my local LAN, and my non-admin accounts from anywhere else. |
I'm going to have to throw in with this idea. From an enterprise security standpoint I can't sacrifice functionality for security if it means the primary purpose of the application cannot be fulfilled. Restricting down accounts with admin permissions is a must, restricting a traveling sales engineer will result in serious issues since they go from hotel to hotel and there is no way to predict what network he will be hitting the server from on login. |
This is very userful , |
Can we expect this for specific user/group anytime soon. It would be very useful. Thanks and Regards |
I would like to put emphasis on this. We would love the ability to restrict administrators to only be able to login within the internal network, |
I'm interested in this as well, specifically for admin/service accounts. For those it would make since to say they could only log in from the local ip range, where other users have either no restriction or their own custom restrictions as well. I think that would make this app a lot more useful. |
This feature is required, access rule on file does only cover file access and is limited (no OR rule). |
This feature would definitely be great to have! |
Contributions are welcome - this app is pretty simple, it should not be terribly hard to do a PR that adds this functionality. In the mean time, thanks for sharing the idea - please refrain from posting more "me too" replies, better use the Github feature to up-vote a comment or something to signal your agreement. Otherwise this gets very long ;-) |
Agree this will be a very desirable enhancement . also a comment line for each IP address to associate |
+1 |
+1 |
any update if this would be supported for latest nextcloud version. |
I was looking for something like this Authorised Networks. |
+1 here. This definitely must have feature of this app. We would like to apply this for admin accounts making it available only for local network or vpn connection. |
Do you really need a table? Seems you might be able to provide a lot of functionality by enriching the config value format. Perhaps add a couple of options to the start of each ip address: ! indicates a blacklist, ie !192.168.1.0/8 means anything but that range. Evaluate left to right, action on first match(ish). Examples: Backward compatibility. Allow access to all from local network and host Allow access to a restricted group only from the local network, everyone else can connect from outside Similarly, allowing access to restricted IDs from the internet but defaulting local otherwise: Is that maybe something you could parse and walk within IsRequestAllowed? |
Greetings,
Being able to enable the plugin only for specific users/groups would be useful.
The text was updated successfully, but these errors were encountered: