Skip to content

Send the pong response packet immediately.#12242

Merged
Spottedleaf merged 1 commit into
PaperMC:mainfrom
0-x-2-2:send_pong_response_immediate
Mar 8, 2025
Merged

Send the pong response packet immediately.#12242
Spottedleaf merged 1 commit into
PaperMC:mainfrom
0-x-2-2:send_pong_response_immediate

Conversation

@0-x-2-2
Copy link
Copy Markdown
Contributor

@0-x-2-2 0-x-2-2 commented Mar 8, 2025

This fixes the F3+3 ping graph in the Minecraft client being offset by 25-50ms.

…the keep alive packet and is processed async.
@0-x-2-2 0-x-2-2 requested a review from a team as a code owner March 8, 2025 02:50
@github-project-automation github-project-automation Bot moved this to Awaiting review in Paper PR Queue Mar 8, 2025
@github-project-automation github-project-automation Bot moved this from Awaiting review to Awaiting final testing in Paper PR Queue Mar 8, 2025
@Spottedleaf Spottedleaf merged commit 0a4eb8f into PaperMC:main Mar 8, 2025
@github-project-automation github-project-automation Bot moved this from Awaiting final testing to Merged in Paper PR Queue Mar 8, 2025
@0-x-2-2 0-x-2-2 deleted the send_pong_response_immediate branch March 8, 2025 05:14
@ghost
Copy link
Copy Markdown

ghost commented Mar 8, 2025

man is it just me that thinks this whole packet queuing thing was a bad idea?

@treshchev
Copy link
Copy Markdown

bro, are you resting at all?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

3 participants