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

Failing to clone entity metadata packets in 1.14, 1.14.1, 1.4.2. Latest ProtocolLib dev version. #637

Closed
4 tasks done
Niall7459 opened this issue Jun 21, 2019 · 12 comments
Closed
4 tasks done
Assignees
Labels

Comments

@Niall7459
Copy link

Niall7459 commented Jun 21, 2019

Follow this template except for feature requests. Use pastebin when providing /protocol dump and any relevant errors.

Make sure you've done the following:

  • You're using the latest build for your server version
  • This isn't an issue caused by another plugin
  • You've checked for duplicate issues
  • You didn't use /reload

Debug paste link:
Multiple:
https://pastebin.com/M5zqwUkd
https://pastebin.com/PAt8MCjU
https://pastebin.com/cDfU1rwc

Description and relevant errors:

Issue cloning PacketPlayOutEntityMetadata. I'm pretty sure this isn't an issue on my side. Any information for anything that needs changing for 1.14 would be great. Thank you.

Packet Event Listener:

    @Override
    public void onPacketSending(PacketEvent event) {
        PacketContainer packet = event.getPacket();

        if (packet.getType() != PacketType.Play.Server.ENTITY_METADATA) {
            return;
        }

        WrapperPlayServerEntityMetadata entityMetadataPacket = new WrapperPlayServerEntityMetadata(packet.deepClone());
        List<WrappedWatchableObject> dataWatcherValues = entityMetadataPacket.getEntityMetadata();

        for (WrappedWatchableObject watchableObject : dataWatcherValues) {
            if (watchableObject.getIndex() == 2) {
                if (replacePlaceholders(watchableObject, event.getPlayer())) {
                    event.setPacket(entityMetadataPacket.getHandle());
                }
                return;
            }
        }
    }
@dmulloy2
Copy link
Owner

Multicraft is cutting out the useful part of the stack:
08.06 00:04:44 [Multicraft] Skipped 76 lines due to rate limit (30/s)

Can you get a complete one?

@HexedHero
Copy link

@Brokkonaut
Copy link
Contributor

I have something similar: https://pastebin.com/raw/hBcMJd1t

@dmulloy2
Copy link
Owner

dmulloy2 commented Jul 1, 2019

Thanks! I'll fix those when I get the chance, hopefully soon-ish

@Niall7459
Copy link
Author

Hey, just wondering if there’s any progress on this or when we can see the fix implemented?

Regards

@WulfTheGod
Copy link

Spams this error multiple times in console.
https://pastebin.com/rXijkPSx
Only happens when players interact with HDs.

@HexedHero
Copy link

Please fix?

@dmulloy2
Copy link
Owner

dmulloy2 commented Aug 3, 2019

Sorry, was really busy the last few months. Working on it now.

@HexedHero
Copy link

Sorry, was really busy the last few months. Working on it now.

Finally, thank the lord !

@successed
Copy link

WHY CLOSED İSSUE ?

http://bin.shortbin.eu:8080/QggbCzj5hk

@HexedHero
Copy link

WHY CLOSED İSSUE ?

http://bin.shortbin.eu:8080/QggbCzj5hk

It's literally fixed, go update.

@WulfTheGod
Copy link

I am updated with everything and the issue still persists.
https://pastebin.com/LM1xdpR0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants