WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@ManInMyVan
Copy link
Contributor

@ManInMyVan ManInMyVan commented Mar 23, 2025

adds an option to queue potential duplicates until the next (non-async) packet to check if it is actually a duplicate before processing, making duplicate abuse slightly harder.

problems:
sometimes causes players to get kicked because of exceptions
require-same-rotation-in-duplicates causes issues when standing still

@SamB440
Copy link
Contributor

SamB440 commented Mar 23, 2025

I already tried this in my original pr but it just causes problems

@ManInMyVan
Copy link
Contributor Author

I already tried this in my original pr but it just causes problems

such as?

@ManInMyVan ManInMyVan force-pushed the require-use-after-duplicate branch from 47140bf to 62c81b3 Compare April 26, 2025 19:43
@ManInMyVan ManInMyVan added the status: rebase required The pull request needs rebasing onto the merge branch label May 4, 2025
@ManInMyVan ManInMyVan force-pushed the require-use-after-duplicate branch from 62c81b3 to eaf09d3 Compare July 7, 2025 02:20
@CLAassistant
Copy link

CLAassistant commented Jul 7, 2025

CLA assistant check
All committers have signed the CLA.

@ManInMyVan ManInMyVan removed the status: rebase required The pull request needs rebasing onto the merge branch label Jul 7, 2025
@ManInMyVan ManInMyVan marked this pull request as draft July 7, 2025 17:24
@Axionize Axionize force-pushed the 2.0 branch 3 times, most recently from 5bef6b9 to 0374b8c Compare August 9, 2025 01:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants