Added role-based restrictions for editing server connections. #1947
parent
2e072baf72
commit
814250aade
Binary file not shown.
|
Before Width: | Height: | Size: 124 KiB After Width: | Height: | Size: 121 KiB |
|
|
@ -719,6 +719,7 @@ class ServerNode(PGChildNodeView):
|
|||
return make_json_response(success=1,
|
||||
info=gettext("Server deleted"))
|
||||
|
||||
@permissions_required(AllPermissionTypes.object_register_server)
|
||||
@pga_login_required
|
||||
def update(self, gid, sid):
|
||||
"""Update the server settings"""
|
||||
|
|
|
|||
|
|
@ -141,6 +141,7 @@ define('pgadmin.browser.node', [
|
|||
function() {
|
||||
return !!(self.canEdit(...arguments));
|
||||
} : (!!self.canEdit),
|
||||
permission: self.type == 'server' ? AllPermissionTypes.OBJECT_REGISTER_SERVER : undefined,
|
||||
}]);
|
||||
}
|
||||
|
||||
|
|
@ -161,7 +162,7 @@ define('pgadmin.browser.node', [
|
|||
function() {
|
||||
return !!(self.canDrop(...arguments));
|
||||
} : (!!self.canDrop),
|
||||
permission: self.type == 'server' ? 'object_register_server' : undefined,
|
||||
permission: self.type == 'server' ? AllPermissionTypes.OBJECT_REGISTER_SERVER : undefined,
|
||||
}]);
|
||||
|
||||
if (self.canDropCascade) {
|
||||
|
|
|
|||
|
|
@ -48,12 +48,12 @@ class PgAdminPermissions:
|
|||
self.add_permission(
|
||||
AllPermissionCategories.object_explorer,
|
||||
AllPermissionTypes.object_register_server,
|
||||
gettext("Register/remove server")
|
||||
gettext("Manage Server")
|
||||
)
|
||||
self.add_permission(
|
||||
AllPermissionCategories.tools,
|
||||
AllPermissionTypes.tools_query_tool,
|
||||
gettext("Query tool")
|
||||
gettext("Query Tool")
|
||||
)
|
||||
self.add_permission(
|
||||
AllPermissionCategories.tools,
|
||||
|
|
@ -63,32 +63,32 @@ class PgAdminPermissions:
|
|||
self.add_permission(
|
||||
AllPermissionCategories.tools,
|
||||
AllPermissionTypes.tools_psql_tool,
|
||||
gettext("PSQL tool")
|
||||
gettext("PSQL Tool")
|
||||
)
|
||||
self.add_permission(
|
||||
AllPermissionCategories.tools,
|
||||
AllPermissionTypes.tools_backup,
|
||||
gettext("Backup tool (including server and globals)")
|
||||
gettext("Backup Tool (including server and globals)")
|
||||
)
|
||||
self.add_permission(
|
||||
AllPermissionCategories.tools,
|
||||
AllPermissionTypes.tools_restore,
|
||||
gettext("Restore tool")
|
||||
gettext("Restore Tool")
|
||||
)
|
||||
self.add_permission(
|
||||
AllPermissionCategories.tools,
|
||||
AllPermissionTypes.tools_import_export_data,
|
||||
gettext("Import/export data")
|
||||
gettext("Import/Export Data")
|
||||
)
|
||||
self.add_permission(
|
||||
AllPermissionCategories.tools,
|
||||
AllPermissionTypes.tools_import_export_servers,
|
||||
gettext("Import/export servers")
|
||||
gettext("Import/Export Servers")
|
||||
)
|
||||
self.add_permission(
|
||||
AllPermissionCategories.tools,
|
||||
AllPermissionTypes.tools_search_objects,
|
||||
gettext("Search objects")
|
||||
gettext("Search Objects")
|
||||
)
|
||||
self.add_permission(
|
||||
AllPermissionCategories.tools,
|
||||
|
|
@ -98,32 +98,32 @@ class PgAdminPermissions:
|
|||
self.add_permission(
|
||||
AllPermissionCategories.tools,
|
||||
AllPermissionTypes.tools_schema_diff,
|
||||
gettext("Schema diff")
|
||||
gettext("Schema Diff")
|
||||
)
|
||||
self.add_permission(
|
||||
AllPermissionCategories.tools,
|
||||
AllPermissionTypes.tools_grant_wizard,
|
||||
gettext("Grant wizard")
|
||||
gettext("Grant Wizard")
|
||||
)
|
||||
self.add_permission(
|
||||
AllPermissionCategories.tools,
|
||||
AllPermissionTypes.tools_erd_tool,
|
||||
gettext("ERD tool")
|
||||
gettext("ERD Tool")
|
||||
)
|
||||
self.add_permission(
|
||||
AllPermissionCategories.storage_manager,
|
||||
AllPermissionTypes.storage_add_folder,
|
||||
gettext("Add folder")
|
||||
gettext("Add Folder")
|
||||
)
|
||||
self.add_permission(
|
||||
AllPermissionCategories.storage_manager,
|
||||
AllPermissionTypes.storage_remove_folder,
|
||||
gettext("Delete file/folder")
|
||||
gettext("Delete File/Folder")
|
||||
)
|
||||
self.add_permission(
|
||||
AllPermissionCategories.miscellaneous,
|
||||
AllPermissionTypes.change_password,
|
||||
gettext("Change password")
|
||||
gettext("Change Password")
|
||||
)
|
||||
|
||||
def add_permission(self, category: str, permission: str, label: str):
|
||||
|
|
|
|||
Loading…
Reference in New Issue