Skip to content

Add more bypass permission nodes. - #13294

Open
Mickey42302 wants to merge 12 commits into
PaperMC:mainfrom
Mickey42302:bypass
Open

Add more bypass permission nodes.#13294
Mickey42302 wants to merge 12 commits into
PaperMC:mainfrom
Mickey42302:bypass

Conversation

@Mickey42302

Copy link
Copy Markdown
Contributor

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")

@Mickey42302
Mickey42302 requested a review from a team as a code owner November 11, 2025 02:41
@github-project-automation github-project-automation Bot moved this to Awaiting review in Paper PR Queue Nov 11, 2025
@electronicboy

Copy link
Copy Markdown
Member

This should not be feature patches, but a lot of this looks implemented improperly and dangerous

@Mickey42302

Copy link
Copy Markdown
Contributor Author

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.

@electronicboy

Copy link
Copy Markdown
Member

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

@Mickey42302

Copy link
Copy Markdown
Contributor Author

What do you feel would be a better approach to implement the permission nodes?

@Mickey42302

Copy link
Copy Markdown
Contributor Author

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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Awaiting review

Development

Successfully merging this pull request may close these issues.

2 participants