Paper: teleporting out of world crashes server

[stupendousMan]

Link/paste of stack trace

___Please do not make reports when the report says "DO NOT REPORT THIS TO PAPER". These are simply messages informing you of lag, to find what is causing it. Ask for tips on Discord or IRC instead.___

Plugin list:

Actions to reproduce (if known):

Paper build number:

version
(Implementing API version 1.15.2-R0.1-SNAPSHOT)

[stupendousMan]

This error is very reproducible (at least for me).

[HexedHero]

Don't teleport out of bounds of the max world size then? and for the love of god use Pastebin

[zachbr]

Patient: "Doctor, it hurts when I do this."
Doctor: "Then don't do that!"

I think the answer here is "don't do that".

Vanilla has a function to check if a location is in bounds, in my opinion we should just hook that on the teleport command and decline if it comes back as outside.

[stupendousMan]

I run a server for many people and how do you suggest I prevent players from doing that, besides asking them/banning them? Really, "don't do that" is not the way to maintain a great piece of software! I like the idea of hooking to a vanilla function much better.

[zachbr]

Yes that first part was more of a joke.

[stupendousMan]

It was a joke for you but maybe not for @HexedHero

[A248]

This issue is no joke. Servers using the vanilla TP command can't have moderators crashing it.

[HexedHero]

For a solution I just tested, you can use the WorldBorder plugin and set a border on the worlds, whatever you like or the same as vanilla's Worldborder and the Worldborder plugin will intercept any teleport commands past the border thus not crashing.

[A248]

I don't use the vanilla TP command myself, but I would wager that someone using vanilla TP wouldn't want to install WorldBorder just to "patch" a bug that should have been fixed at the source. This isn't hard to fix either. It's just argument validation.

[HexedHero]

Well, I provided a temp solution for you, if you don't wanna use something like that to patch it until it's fixed then enjoy the crashes I guess.

[HexedHero]

If you care
/wb set 29999999 29999999 0 0

with WB plugin sets the border at the vanilla world end and stops the crashes.

[electronicboy]

This argument/debate/whatever is not needed

[HexedHero]

Isn't even argument/debate. Just provided a solution to help a fellow user until when/if it gets fixed. Should be posted on Mojang JIRA since its using vanilla commands etc.