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

Admin Role Users Without Permission Can Change Primary Allocation #600

Closed
3 tasks done
FordCorn opened this issue Oct 2, 2024 · 1 comment · Fixed by #601
Closed
3 tasks done

Admin Role Users Without Permission Can Change Primary Allocation #600

FordCorn opened this issue Oct 2, 2024 · 1 comment · Fixed by #601
Labels
🐛 bug Something isn't working 🟢 easy Good for newcomers

Comments

@FordCorn
Copy link

FordCorn commented Oct 2, 2024

Current Behavior

Users given an admin role that only allows servers to be viewed are still able to change the primary allocation even without permission.

Extra info: the user itself is also not able to add, remove, or modify current allocations on the server either, only view.

Expected Behavior

The user should not be able to change the default allocation without the permission to change allocations on a server.

Steps to Reproduce

Create a user role in the admin panel and give it server view permissions, log in as said user and look at the servers page, the user is able to change the primary allocation even though they do not have the required permissions to do so, or if this is even intended.

Panel Version

1.0.0-beta10

Wings Version

1.0.0-beta5

Games and/or Eggs Affected

Minecraft (Fabric)

Docker Image

ghcr.io/parkervcp/yolks:java_21

Error Logs

No response

Is there an existing issue for this?

  • I have searched the existing issues before opening this issue.
  • I have provided all relevant details, including the specific game and Docker images I am using if this issue is related to running a server.
  • I have checked in the Discord server and believe this is a bug with the software, and not a configuration issue with my specific system.
@RMartinOscar
Copy link
Contributor

Hey should be fixed with #601 thanks for your report !

@Boy132 Boy132 added 🐛 bug Something isn't working 🟢 easy Good for newcomers labels Oct 2, 2024
@Boy132 Boy132 linked a pull request Oct 2, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working 🟢 easy Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants