PluginBugs / Issues-ItemsAdder

Repository used to keep track of issues of my plugin ItemsAdder
https://itemsadder.devs.beer
54 stars 21 forks source link

[BUG] server crashing #1146

Closed 100petr closed 3 years ago

100petr commented 3 years ago

Describe the bug If I came to this machine (schematic), the server always crash on purpur, not on the others (tuinity, paper, spigot). logs: https://discord.com/channels/@me/550357534066868255/881856321828311090 schematic: https://discord.com/channels/@me/550357534066868255/881858418724458527

To Reproduce Steps to reproduce the behavior:

  1. Go to the machine (schem)
  2. server crash

Expected behavior the server will not crash

Screenshots image

Info (please complete the following information):

Special Info Please do not write "latest"! Tell us the specific plugin version you are using.

Additional context I don't know if this is a direct IA problem, you wanted to send it.

LoneDev6 commented 3 years ago

By reading the log:

Seems that for some reason Purpur is calling the BlockPhysics update when ItemsAdder calls this line: p.playNote(e.getBlock().getLocation(), Instrument.getByType(instrument), data.getNote()); Or maybe calls NotePlayEvent for some reason, and this creates a chain effect since I'm calling the provided code in a NotePlayEvent listener. This doesn't happen on Paper or Spigot.

Doing a test:

Nothing bad happens, the "music" plays correctly and no crash happens.

image

Tested on

[18:42:28 INFO]: This server is running Purpur version git-Purpur-1332 (MC: 1.17.1) (Implementing API version 1.17.1-R0.1-SNAPSHOT) (Git: 1105e10 on ver/1.17.1)
[18:42:28 INFO]: You are 38 version(s) behind
[18:46:37 INFO]: This server is running Purpur version git-Purpur-1370 (MC: 1.17.1) (Implementing API version 1.17.1-R0.1-SNAPSHOT) (Git: cdbf59d on ver/1.17.1)
[18:46:37 INFO]: You are running the latest version

Solution

I have no solution for this issue, I can't manage to trigger it. Maybe it's something in your config of purpur.yml? You can send it to me if you want.

100petr commented 3 years ago

Yes, it's something with that purpur.yml and it never occurred to me. If I let it regenerate it works normally. https://discord.com/channels/@me/550357534066868255/883667921845489676

100petr commented 3 years ago

Its due blocks.disable-note-block-updates: true

100petr commented 3 years ago

https://github.com/pl3xgaming/Purpur/commit/e444ad7e616ddbcd2827d2f002fb6d7b560cdb3b