IntellectualSites / IntellectualSites/PlotSquared

Server Crash whilst deleting a plot

Open
#4,337 4 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Bug Requires Testing
Dominant language
Java
Stars
653
Forks
975
Avg merge
2m
Merged PRs (30d)
6

Description

Server Implementation

Paper

Server Version

1.20.4

Describe the bug

When deleting a plot filled with Sculk Sensors and Trapdoors the Server crashes.(Gist file 1) After restarting the server and retrying the server the console is flooded with lots and lots of errors. (Found in the gist. File 2). The fps also drops below 2.

This is an error with Plotsquared and not with the normal paper implementation as it does not occur when removing the blocks with Worldedit (FAWE), or vanilla commands. It only occurs when deleting the plot with plotsquared. As far as I tested.

The size of the plot is not important. The error seemingly only occurs if the plot is deleted whilst a sculk Sensor is deactivating and updating the blocks near it

To Reproduce
  1. Fill a layer of the plot with sculk sensors.
  2. Place a layer of Trapdoors on top of it.
  3. Add an Observer clock to always pulse on of the trapdoors. This sets up an chain reaction of opening and closing shulkers.
  4. Delete the plot (/p delete)
Expected behaviour

The plot should be deleted without crashing the Server ;)

Screenshots / Videos

image

Error log (if applicable)

https://gist.github.com/Hasenzahn1/07edcf4b9193df2b3cde7d4599a5717d

Plot Debugpaste

https://athion.net/ISPaster/paste/view/901e76a736af41c1a03b0e41536d6f7e

PlotSquared Version

Plotsquared-bukkit-7.3.2-Premium

Checklist
Anything else?

No response

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing the crash on Paper 1.20.4 with the listed Sculk Sensor, Trapdoor, and Observer setup, then inspect the linked error log and Plot debugpaste while tracing /p delete. Done means deleting the plot completes without a server crash, error flood, or severe FPS drop.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
backend, game-dev
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.