Check Line of Sight of Player Before Block Placements and Opening Containers

[Axionize]

This prevents players from using Freecam and interacting with blocks through walls. For example, accessing a chest behind a wall they shouldn't have access to.

Can be used to check all block interactions, but currently only checks containers for maximum performance. The list of blocks being interacted with to check will be configurable to all or a list.

This also prevents laggy players from being able to break a block in regions protected by plugins such as Worldguard and quickly opening a container behind it before the block reappears client side.

Currently making a PR To ask for feedback on:

• What changes, if any, need to be made for latency compensation.
• If suggested configuration options are appropriate
• What the result of using the Player.BLOCK_INETRACTION_RANGE attribute will have for players on < 1.20.5
• Note that the existing code has comments and print statements to make it more clear what's going on that may not be in the final PR.

EDIT: This could have the potential to close #1498

[ManInMyVan]

this should be done in rotationplace, but I added a review anyways (I gave up halfway through so some things don't have comments)

[Anthony01M]

This needs a lot of rework and seems like is still in draft, please change the status from Open to Draft

[Axionize]

I wrote this as a quick port of a hacky spigot-api based check I had in a custom plugin for a very specific problem on a specific server, which I wanted to move to a proper anticheat.

I'm not that familiar with GrimAC's codebase, but I noticed that Sam was working on something similar: #1238

I cleaned it up a little here: https://github.com/Axionize/Grim. I'm going take to modifying Sam's check to also work for Blocks/Entities/Digging in the reach-wall-check branch and continue providing dev builds with the reach check's and latest GrimAC updates in the 2.0 branch in my fork.

I plan to make a separate draft for this new approach when I'm ready to get some feedback.

I greatly appreciate your time and help, and I'd still like for the code to be reviewed. If there's anything I can do to make that easier, please let me know.

[SamB440]

The problem with my draft is that it falses due to the movement threshold, you could be anywhere within 0.03 and thus the raytrace is wrong

[Axionize]

Wasn't that bug fixed past 1.18.2?

I've taken a look at how you account for this movement and I don't see anything obviously wrong with it? Are there any reproducible circumstances under which you can cause it to fail I can investigate?

In any case, I don't see anything wrong with disabling the 0.03 offsets if the player's version is > 1.18.2. It would be better than nothing and most modern (non 1.7/1.8) servers only keep support for the last couple of major MC versions anyways.

Please let me know if I'm overlooking something or you have a better alternative in mind.

[ManInMyVan]

Wasn't that bug fixed past 1.18.2?

no it's still a thing, it's just 0.0002 and not 0.03, it's just called "0.03" because it's the same mechanic

[Dg32z]

2024-10-11.21-04-19.mp4

false flag

[Axionize]

Client and server version? What block is the plant on the right?

[Dg32z]

Quote reply
Reference in n

1.21.1 Folia
1.21.1 vailla
coral

[Dg32z]

Client and server version? What block is the plant on the right?

2024-10-14.12-56-44.mp4

2024-10-14.12-54-26.mp4

[Dg32z]

Client and server version? What block is the plant on the right?

2024-10-14.12-56-44.mp4
2024-10-14.12-54-26.mp4

ver
[13:11:23 INFO]: Checking version, please wait...
[13:11:24 INFO]: This server is running Luminol version 1.21.1-DEV-dev/1.21.1@791d374 (2024-08-31T08:19:01Z) (Implementing API version 1.21.1-R0.1-SNAPSHOT)
You are 13 version(s) behind
Download the new version at: https://github.com/LuminolMC/Luminol
Previous version: 1.21.1-DEV-7c312e7 (MC: 1.21.1)
pl
[13:11:24 INFO]: Server Plugins (5):
[13:11:24 INFO]: Bukkit Plugins:
[13:11:24 INFO]: - GrimAC, LiteBans, LuckPerms, PlaceholderAPI, PlugManX

[Dg32z]

2024-10-14.13-12-52.mp4