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

Minecraft 1.21.5 #1489

Merged
merged 17 commits into from
Mar 27, 2025
Merged

Minecraft 1.21.5 #1489

merged 17 commits into from
Mar 27, 2025

Conversation

RealBauHD
Copy link
Contributor

No description provided.

@4drian3d 4drian3d added the status: in progress Work in progress label Jan 16, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@pkt77
Copy link
Contributor

pkt77 commented Mar 17, 2025

Bug: Players kicked for chat packets. This is due to the new byte checksum at the end of LastSeenMessages

Gerrygames and others added 4 commits March 18, 2025 17:03
@RealBauHD
Copy link
Contributor Author

Before we can actually merge this, we need something to respect the component changes e. g. KyoriPowered/adventure#1168

@electronicboy
Copy link
Member

I fixed some issues connecting to the latest RC in this, however, there still appears to be something going sideways for me, and, I need to get back to being productive

@RealBauHD
Copy link
Contributor Author

What kind of problems have you encountered?

@electronicboy
Copy link
Member

First I was getting kicked because of the argument registry being incorrect,

Now;

---- Minecraft Network Protocol Error Report ----
// Don't worry, I'll be fine

Time: 2025-03-25 12:53:56
Description: Packet handling error

java.lang.IllegalStateException: Registry minecraft:advancement not found
	at ji$a.i(SourceFile:116)
	at java.base/java.util.Optional.orElseThrow(Optional.java:403)
	at ji$a.e(SourceFile:116)
	at fp.<init>(SourceFile:38)
	at fp$a$a.a(SourceFile:113)
	at fp$a$a.b(SourceFile:104)
	at acx$a.a(SourceFile:170)
	at acx$d.a(SourceFile:336)
	at acx$d.a(SourceFile:348)
	at acx$d.a(SourceFile:348)
	at acx$d.a(SourceFile:348)
	at acx$d.a(SourceFile:348)
	at acx$d.a(SourceFile:348)
	at acx.a(SourceFile:266)
	at glp.a(SourceFile:1557)
	at acx.a(SourceFile:262)
	at acx.a(SourceFile:39)
	at zr.a(SourceFile:27)
	at btq.d(SourceFile:164)
	at btv.d(SourceFile:23)
	at btq.B(SourceFile:138)
	at btq.bA(SourceFile:123)
	at fqq.c(SourceFile:1308)
	at fqq.f(SourceFile:936)
	at net.minecraft.client.main.Main.main(SourceFile:265)
	at org.prismlauncher.launcher.impl.StandardLauncher.launch(StandardLauncher.java:105)
	at org.prismlauncher.EntryPoint.listen(EntryPoint.java:129)
	at org.prismlauncher.EntryPoint.main(EntryPoint.java:70)

I'm imagining that there is still some issue with arg parsing and it's tripping up over on consuming something it shouldn't, I'd probably want to double check the one I added, but, I'm not sure

@RealBauHD
Copy link
Contributor Author

Should be fixed now, the error was caused by the class being registered twice, resulting in an incorrect class to id mapping.

@electronicboy
Copy link
Member

hnnnnng; I really want to find some means of cleaning this system up; debugging when stuff goes sideways is often such a pain in the arse

@PedroMPagani
Copy link

Maybe it would be nice to remove that blockage on the write packet for the chat queue

@kennytv kennytv marked this pull request as ready for review March 27, 2025 14:55
@kennytv
Copy link
Member

kennytv commented Mar 27, 2025

Ready for review I decided:tm: the rest can be handled later

@kennytv kennytv merged commit d2cd791 into PaperMC:dev/3.0.0 Mar 27, 2025
1 check passed
pull bot pushed a commit to WiIIiam278/Velocity that referenced this pull request Mar 27, 2025
Missing adventure component changes, so entity and item hovers from the API may not work for 1.21.5 clients

Co-authored-by: Gero <[email protected]>
Co-authored-by: Shane Freeder <[email protected]>
@4drian3d 4drian3d removed the status: in progress Work in progress label Mar 30, 2025
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

7 participants