Skip to content
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

Hide SOL Console Button on Overview Based on User Privilege #124

Open
FarahRasheed1 opened this issue Jul 9, 2024 · 1 comment
Open
Labels
enhancement New feature or request

Comments

@FarahRasheed1
Copy link
Contributor

Describe the bug

The SOL (Serial Over LAN) console button is currently visible to all users on the Overview screen, regardless of their privilege level. This can lead to unauthorized access to the SOL console.

To Reproduce Steps to reproduce the behavior:

  1. Log in to the system with a user account that does not have the necessary privileges for accessing the SOL console (privilege role of operator or read only).
  2. Navigate to the 'Overview' screen.
  3. Observe that the SOL console button is visible and accessible.

Expected behavior

The SOL console button should be conditionally rendered in the Overview page based on user privilege. Only users with the appropriate permissions should see and access this button. This ensures that unauthorized users cannot access the SOL console functionality.

Screenshots

image

Additional context

This bug affects the visibility of the SOL console button in the OverviewQuickLinks component. Proper privilege checks should be implemented to control access to this functionality.

@FarahRasheed1
Copy link
Contributor Author

FarahRasheed1 commented Jul 9, 2024

Bench Test Results:

The SOL console button is visible to admin users:

PostFixBenchTestAdministratorPrivelege

The SOL console button is hidden from non-admin users:

PostFixBenchTestNonAdministratorPrivelege

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants