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

Pass handler to clientside ConfigurationPacketHandler #3305

Closed

Conversation

apple502j
Copy link
Contributor

Breaking change, needs to be merged ASAP.

Turns out you cannot query CCNH from MinecraftClient. We first need to get ClientConnection (which is also PITA - the impl already does that elsewhere), then get the handler through Netty internals and several casting.

Also adds a helper method to disconnect, as there is currently no way to do so. I wasn't sure if interface injection or static method was better; you can revert the commit if this needs to be discussed later.

@apple502j apple502j added enhancement New feature or request fabric-networking Pull requests and issues related to the networking api priority:high High priority PRs that need review and work now. Review these first. labels Sep 8, 2023
@apple502j apple502j requested review from modmuss50 and a team September 8, 2023 05:07
@Technici4n
Copy link
Member

Question: why is the handler useful?

@apple502j
Copy link
Contributor Author

Closing, on second thought this doesn't look like it's needed in many use cases.

@apple502j apple502j closed this Sep 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request fabric-networking Pull requests and issues related to the networking api priority:high High priority PRs that need review and work now. Review these first.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants