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

Server Spoof and Packet Canceller do not work as expected #4593

Closed
2 tasks done
CyberFoxar opened this issue May 21, 2024 · 5 comments · Fixed by #4595
Closed
2 tasks done

Server Spoof and Packet Canceller do not work as expected #4593

CyberFoxar opened this issue May 21, 2024 · 5 comments · Fixed by #4595
Labels
bug Something isn't working.

Comments

@CyberFoxar
Copy link

CyberFoxar commented May 21, 2024

Describe the bug

When using the "Server Spoof" module, the client still sends unmodified packets to the server.
On the same vein, the "Packet Canceller" module does not block any kind of packets.

This seems to stems from an incorrect pointcut/mixin usage around the ClientConnectionMixin. Looking over at what Wurst does (which does not have this issue), they seem to do doing their injection a bit differently than how meteor does. Here's the file in question on their repo. -- Though, as of Wurst 7.42, it does not seem to work either.

Sadly, my java-fu is not high enough to suggest a fix, but I can point out this.

Steps to reproduce

  1. Install Meteor client
  2. Install any kind of mod that send stuff on the minecraft:register channel. Mods like VoxelMap and Controlify are good candidates.
  3. Add a MitM proxy to examine in/out packets, for example pakkit
  4. Launch minecraft, on main menu and either:
  • Enable Packet Canceller and block C2S packet: CustomPayloadC2SPacket
  • Enable Server Spoof and enable channel block with the default minecraft:register value
  1. Connect to any server using the MitM proxy
  2. Observe that CustomPayloadC2SPackets are still sent and that they even have the minecraft:register channel.

Meteor Version

0.5.7-2079

Minecraft Version

MC 1.20.6

Operating System

Windows

Before submitting a bug report

  • This bug wasn't already reported (I have searched bug reports on GitHub).

  • This is a valid bug (I am able to reproduce this on the latest dev build).

@CyberFoxar CyberFoxar added the bug Something isn't working. label May 21, 2024
@19MisterX98
Copy link
Contributor

image
meteor injects send events in the upper method while wurst injects in the lower one.

fabrics networking lib calls the lower method and therefore bypasses the event system in meteor.
I'll fix it

@CyberFoxar
Copy link
Author

Fast ! I'll check it out when the dev build comes out.

@Wide-Cat
Copy link
Collaborator

Shoutout to @19MisterX98

@CyberFoxar
Copy link
Author

CyberFoxar commented May 24, 2024

Weirdly enough, it still seems to happen with version 0.5.7-2079.
Having a MitM proxy and Controlify and packetCanceller or server spoof active... they do not seem to have any effect.

I mean, server spoof does correctly report minecraft:brand as vanilla but it does not block mincraft:register channels, or any kind of channels for that matter.

Packet Canceller does not seems to cancel any packets either.

@CyberFoxar
Copy link
Author

Nevermind, I was using an old build. It's working as expected now !

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

Successfully merging a pull request may close this issue.

3 participants