Repository navigation
fix(rbac): replace dashboard super-admin-only access with dashboard.view permission - #424
Conversation
…iew permission
The dashboard was restricted to super administrators because the old
dashboard displayed sensitive connection logs. After the dashboard
refactor the content was streamlined to overview statistics only, but
the permission was never updated. Add a dedicated 'dashboard.view'
permission code to the RBAC catalog and replace @RequireSuperAdmin()
with @RequirePermission('dashboard.view') so that roles with this
permission can access the dashboard.
Closes #415
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Walkthrough
Merge Risk: ⚪ Minimal · up to A super administrator can explicitly grant dashboard access to roles. The dashboard endpoints return overview and trend aggregates rather than per-connection logs. Pre-merge checks |
|
Summary
dashboard.viewpermission code to the RBAC catalog@RequireSuperAdmin()with@RequirePermission('dashboard.view')on the dashboard controllerContext
The dashboard was originally restricted to super administrators because the old dashboard displayed sensitive connection logs. After the dashboard refactor the content was streamlined to overview statistics only, but the permission was never updated. This means that even roles with all permissions (e.g. System Administrator) cannot access the dashboard — only the super admin can.
This PR introduces a
dashboard.viewpermission so that any role granted this permission can access the dashboard.Closes #415
Summary by CodeRabbit