The following warnings occurred:
Warning [2] Undefined variable $unreadreports - Line: 26 - File: global.php(961) : eval()'d code PHP 8.1.2-1ubuntu2.14 (Linux)
File Line Function
/global.php(961) : eval()'d code 26 errorHandler->error
/global.php 961 eval
/showthread.php 28 require_once





× This forum is read only. As of July 23, 2019, the UserSpice forums have been closed. To receive support, please join our Discord by clicking here. Thank you!

  • 0 Vote(s) - 0 Average
  • 1
  • 2
  • 3
  • 4
  • 5
master_account and the backup feature
#1
One thing that we might want to consider is that the backup function allows any admin to download the full source code and db of a project that might be closed source. I had to take away several people's admin privileges on my own projects because of this. What are your thoughts on making backups master only? Or maybe if we take away some of the messages, all admins could hit the backup button, but not see any of the logging or download backups?
  Reply
#2
I'm not okay with this as I for security purposes keep my User ID 1 as a placeholder account and ban it, and make a secondary account for myself with full admin access, but no master access. Purely a security thing as I don't want someone to get access to my account and have complete and utter access to every aspect of the system even if I do whatever to lock them out. (eg I sign in from the Dev Account and remove their permission levels, but they still got access bc they are master).

I would use it in the following manner if you were to restrict it:
Jr Admin - Click backup
Sr Admin - Can download and logging

Or in my terms of what I use:
Administrator
Database Admin (would be same as Jr Admin above)
  Reply
#3
We can think through that. Either way, I think we need a little bit of limitation on the backup feature. Also, when we were having problems with the recursive backups, I accidentally filled up my server Smile That's kind of a problem.
  Reply
#4
That is totally a problem Wink You might want to fix that!

I think two admin levels might be best...especially since not everyone is using master account, and clearly it is not always in everyones configurations (as per a recent issue in another thread)
  Reply
#5
Right.

Forcing out a new permission level gets sticky because the permissions are usually hard coded into the php, but I'm sure we can get creative.
  Reply
#6
Why don't we add a variable to the user profiles? That can only be modified based on whatever criteria we set? And if they have a value of whatever (1 I guess) in that spot they have access to all of admin_backup, where if they dont and only have the admin permission, they lose whatever we set
  Reply
#7
What permission level would you recommend be able to set that variable?
  Reply
#8
Master Account would be fine to set this variable. Ultimately you can change this in the DB (change the value) if you needed to give someone permission and something fatal happened or whatever
  Reply


Forum Jump:


Users browsing this thread: 1 Guest(s)