Skip to content

Add Missing Packet Handler ( WeaponRack, SyncItemsWithShimmer, SyncItemCannotBeTakenByEnemies and EntityLeash ) #3271

Open
NightKLP wants to merge 11 commits into
Pryaxis:general-develfrom
NightKLP:general-devel
Open

Add Missing Packet Handler ( WeaponRack, SyncItemsWithShimmer, SyncItemCannotBeTakenByEnemies and EntityLeash ) #3271
NightKLP wants to merge 11 commits into
Pryaxis:general-develfrom
NightKLP:general-devel

Conversation

@NightKLP

Copy link
Copy Markdown
Contributor

it's almost the same as Item Frame function but it works which just notice why this packet doesn't exist yet

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@lost-werewolf lost-werewolf left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great, thanks for the catch on this!

@ACaiCat ACaiCat left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as expected

@NightKLP NightKLP changed the title Add WeaponRack Packet Add Missing Packet Handler ( WeaponRack, SyncItemsWithShimmer and SyncItemCannotBeTakenByEnemies ) Apr 30, 2026
@NightKLP NightKLP changed the title Add Missing Packet Handler ( WeaponRack, SyncItemsWithShimmer and SyncItemCannotBeTakenByEnemies ) Add Missing Packet Handler ( WeaponRack, SyncItemsWithShimmer, SyncItemCannotBeTakenByEnemies and EntityLeash ) Apr 30, 2026
@NightKLP

NightKLP commented Apr 30, 2026

Copy link
Copy Markdown
Contributor Author

In the past days ive been exploited by hackers in my server and decided to investigate it ive build a packet logger ( good thing my server can handle it had so much memory that can handle it )
hackers just spawn platinum coin out of tin air to players which the packet logs shows it was ( SyncItemsWithShimmer and SyncItemCannotBeTakenByEnemies ) apparently

the packets i coded and unsure is:

  • HandleSyncItemCannotBeTakenByEnemies

while this handler is not fully finish will be:

  • TELeashedEntityAnchorPlaceItem

@lost-werewolf lost-werewolf left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you can make Item Drop (21), SyncItemsWithShimmer (145), and SyncItemCannotBeTakenByEnemies (148) use the same args (expanding the Item Drop args) & handling, this should be good once you've finished, as the previous changes were great.

Comment thread TShockAPI/GetDataHandlers.cs Outdated
Comment thread TShockAPI/GetDataHandlers.cs
Added event handler to prevent items from being taken by enemies under certain conditions, including position and velocity checks.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants