Add more bypass permission nodes. - #13294
Conversation
|
This should not be feature patches, but a lot of this looks implemented improperly and dangerous |
|
What makes them dangerous? The same privileges are granted to users with the appropriate level of operator access. These permission nodes are meant to provide a way to give players those privileges without having to use the "/op" command. |
|
What makes them dangerous is that it looks like they're not disabling the limits, they're just blindly returning inside of the limits meaning that the limit still applies, it just doesn't either properly process the interaction or properly revoke it |
|
What do you feel would be a better approach to implement the permission nodes? |
|
Everything is converted to file patches now. I've also made some other adjustments. Is this what you're looking for, or are more changes needed? |
I would like to suggest adding bypass permissions for more built-in vanilla features. The patch I created adds the following nodes:
• Force Gamemode ("minecraft.force-gamemode.bypass")
• Spawn Protection ("minecraft.spawn-protection.bypass")
• Anti-spam ("minecraft.spam.bypass")
• Movement ("minecraft.movement.bypass.vehicle", "minecraft.movement.bypass.client", "minecraft.movement.bypass.elytra")
• Flight ("minecraft.flight.bypass.vehicle" and "minecraft.flight.bypass.client")