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.

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

nether-ceiling-void-damage-height doesn't alter portal search #7208

Open
sudofox opened this issue Dec 27, 2021 · 4 comments · May be fixed by #6986
Open

nether-ceiling-void-damage-height doesn't alter portal search #7208

sudofox opened this issue Dec 27, 2021 · 4 comments · May be fixed by #6986
Labels
good first issue priority: normal status: accepted type: feature

Comments

@sudofox
Copy link

@sudofox sudofox commented Dec 27, 2021 •
edited

Expected behavior

Portals made above the nether roof should not be considered as valid targets as they'll instant kill any player who travel to them when nether-ceiling-void-damage-height is set to the roof - it should try for a different portal instead

Observed/Actual behavior

Player exits above roof, takes void damage, dies, and so do their items.

Steps/models to reproduce

Build portal on roof, then set nether-ceiling-void-damage-height=128

Plugin and Datapack List

NoCheatPlus
IllegalStack
BanItem
Pistonchat

Paper version

[01:27:22 INFO]: This server is running Paper version git-Paper-400 (MC: 1.17.1) (Implementing API version 1.17.1-R0.1-SNAPSHOT) (Git: 4ac995a on ver/1.17.1)
You are 2 version(s) behind
Download the new version at: https://papermc.io/downloads
Previous version: git-Paper-398 (MC: 1.17.1)

Other

No response
@sudofox sudofox added status: needs triage type: bug labels Dec 27, 2021
@sulu5890 sulu5890 added type: feature good first issue priority: normal and removed type: bug status: needs triage labels Dec 28, 2021
@Machine-Maker Machine-Maker linked a pull request Dec 28, 2021 that will close this issue
Open
@stale
Copy link

@stale stale bot commented Feb 26, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@Chew Chew added status: accepted and removed resolution: stale labels Feb 26, 2022
@sudofox
Copy link
Author

@sudofox sudofox commented Feb 27, 2022

Er, did this get fixed(?)

@sudofox
Copy link
Author

@sudofox sudofox commented Mar 13, 2022

I've had several players get killed just by popping out an old portal onto the roof and I can't for the life of me figure out how to get this fixed in paper's current state. We're on the latest 1.17.1 build; not ready to update to 1.18.1 yet due to disk usage concerns.

I'll link them to this issue

@JDButter
Copy link

@JDButter JDButter commented Mar 13, 2022

I just got killed to spawn after flying over 1.5 million blocks on the way to a base. Please fix this.
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue priority: normal status: accepted type: feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix configs that relied on outdated min/max y levels Machine-Maker/Paper
4 participants
@sudofox @Chew @JDButter @sulu5890