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

1.19 protocol (759) #708

Draft
wants to merge 18 commits into
base: master
Choose a base branch
from
Draft

1.19 protocol (759) #708

wants to merge 18 commits into from

Conversation

iceiix
Copy link
Owner

@iceiix iceiix commented Aug 7, 2022

@iceiix
Copy link
Owner Author

iceiix commented Aug 7, 2022

A long ways off, massive changes in this update including to the login handshake: attempting to join fails: disconnect.genericReason, server logs:

[17:46:21] [Server thread/INFO]: /[0:0:0:0:0:0:0:1]:65313 lost connection: Internal Exception: io.netty.handler.codec.DecoderException: java.lang.IndexOutOfBoundsException: readerIndex(12) + length(1) exceeds writerIndex(12): PooledUnsafeDirectByteBuf(ridx: 12, widx: 12, cap: 12)

Has Sig Data, signatures added to https://wiki.vg/index.php?title=Protocol#Login_Start, and more

@iceiix
Copy link
Owner Author

iceiix commented Aug 8, 2022

Now able to login, and can see… something. Flat terrain with a hole is expected, but should be dirt instead of logs, and somehow it is positioned above(?) the player instead of beneath them on the ground:

Screen Shot 2022-08-07 at 6 01 34 PM

This may be related to the min_y changes.

Not all packets parsing is implemented yet, both new packets and changed packets.

Also https://wiki.vg/Protocol_version_numbers says 1.19.1 is 760, but has no protocol change details (yet?). https://minecraft.fandom.com/wiki/Java_Edition_1.19.1 says "This version is not compatible with 1.19 servers.". And another update just dropped, https://minecraft.fandom.com/wiki/Java_Edition_1.19.2 released "released on August 5" two days ago, but it "is compatible with 1.19.1 servers".

@iceiix
Copy link
Owner Author

iceiix commented Aug 8, 2022

Fixed the dimension type loading, which was needed for min_y, now able to see the world as it is:

Screen Shot 2022-08-07 at 6 37 00 PM

but not stable. Walking towards the village crashes with:

thread 'main' panicked at 'TODO: support chunk data y<0 non-air (bit_size 4, single_value Some(4), block_count 4)', src/world/mod.rs:1059:25

which was the hack introduced in #640 (comment) to get past not supporting negative y coordinates - may have to support it.. update: this may be improved with #727 Support chunks with negative y values

@terrarier2111
Copy link
Contributor

terrarier2111 commented Aug 8, 2022

This is great work! Do you think it would be possible to work together to get this patch for leafish as well? I can't work on this rn tho, as i am implementing wgpu-mc as our new renderer, which enables vulkan, dx and metal support for leafish

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants