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

Declined resource pack status not sent when connecting to another server #933

Closed
SamB440 opened this issue Jan 17, 2023 · 2 comments
Closed
Labels
type: bug Something isn't working

Comments

@SamB440
Copy link

SamB440 commented Jan 17, 2023

I have a plugin installed on two backend servers that manage resource packs. It sends a resource pack to the player when they join. This works fine if you join one of these servers directly.

You can connect to server 1 and server 2 directly via a forced host. Let's say, in our server list, we have server 1 set to decline any resource pack requests on our client. We will join server 1. Server 1 receives the status that we have declined the resource pack, as expected.

However, if we now switch to server 2 using /server server-2, server 2 never receives the declined status from our client.

The resourcepack is 100% sent to the client in both instances and I have tested with a client mod that it does indeed receive the resource pack on both servers.

With a plugin on the proxy, I have confirmed that it does receive the response from the client, it is simply not forwarded to the backend server. I've also tried delaying the sending by several seconds, the same problem happens.

Accepted resource pack statuses work as expected, this only happens with declined. In both cases the resource pack is not set to required.

@Xernium
Copy link
Member

Xernium commented Jan 17, 2023

The state machine is a bit iffy here, if you declined the resource pack on the client then Velocity should automatically respond to all requests itself with a “denied” response unless they’re forced- Since forwarding them would be redundant, this is what should happen.

public boolean onResourcePackResponse(PlayerResourcePackStatusEvent.Status status) {
this happens here, though I am not sure where the issue may be

@Nacioszeczek Nacioszeczek added the type: bug Something isn't working label Aug 12, 2023
@4drian3d
Copy link
Contributor

4drian3d commented Feb 8, 2024

Fixed in #1234

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants