Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

You can still crash servers using /p move #4375

Open
2 tasks done
GogleSiteBank opened this issue Mar 16, 2024 · 4 comments
Open
2 tasks done

You can still crash servers using /p move #4375

GogleSiteBank opened this issue Mar 16, 2024 · 4 comments
Labels
Approved A project maintainer or developer has reproduced this issue Bug Something isn't working Priority This issue has priority over backburner issues and should be fixed as soon as possible.

Comments

@GogleSiteBank
Copy link

Server Implementation

Paper

Server Version

1.20.4

Describe the bug

moving plot to big coords = server crash

To Reproduce

  1. /p move bigcoord;bigcoord
  2. crash

Expected behaviour

not crashing

Screenshots / Videos

No response

Error log (if applicable)

https://paste.gg/p/anonymous/f31c4f88e15c4a21894c41bd919cc18f/files/fe86ee018a1b44fbae897a93f7d3ad2d/raw

Plot Debugpaste

https://athion.net/ISPaster/paste/view/cc861d1cdf9d4bf18a4f5cfe1568c599

PlotSquared Version

PlotSquared (Version: PlotSquared-7.3.6-SNAPSHOT)

Checklist

Anything else?

No response
@GogleSiteBank GogleSiteBank added the Requires Testing This is a new issue which needs to be approved before labeled with "bug" label Mar 16, 2024
@OneLiteFeather OneLiteFeather added Bug Something isn't working Priority This issue has priority over backburner issues and should be fixed as soon as possible. Approved A project maintainer or developer has reproduced this issue and removed Requires Testing This is a new issue which needs to be approved before labeled with "bug" labels Mar 16, 2024
@OneLiteFeather
Copy link
Member

@IntellectualSites/core-team please also look into this asap. I replicated it at this version https://athion.net/ISPaster/paste/view/b39d5c49c89e4cd3a56b1537dc60da82 where I was sure this was the version it was fixed (I updated it one day after the fix). Not 100% sure, but a code lookout wouldn't hurt

@PierreSchwang
Copy link
Member

PierreSchwang commented Mar 17, 2024 •
edited

2187497 (X/Z) is generally inside the allowed range of minecraft coordinates. the complete stacktrace would be interesting - as the error log is cutoff after exception:

NVM, it's a chunk coordinate. Should check that there as well, as limited by Paper: PaperMC/Paper#8208

@OneLiteFeather
Copy link
Member

@PierreSchwang
2024-03-16-1_2.log
This is what happened with my version which was one day after the first fix of this issue in /p visit

@OneLiteFeather
Copy link
Member

OneLiteFeather commented Mar 19, 2024 •
edited

Oh well I guess I'll put this in a paste service. I wanted but somehow I can't upload a whole file there xD
Our OneLitePaper is just logging it in json, no other changes to paper

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Approved A project maintainer or developer has reproduced this issue Bug Something isn't working Priority This issue has priority over backburner issues and should be fixed as soon as possible.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@PierreSchwang @OneLiteFeather @GogleSiteBank