Skip to content

Conversation

@DaniGP17
Copy link
Contributor

Goal of this PR

Port the ONESYNC_ENABLE_REMOTE_ATTACHMENT_SANITIZATION native to RedM using the SET_CLIENT_CONFIG_BOOL native.
I think the object is teleporting because the sector pos data node is being updated by the remote client so it "follows" you, but this is the same behavior as FiveM, if you exit the scope of the remote player the object stop following.

2025-12-16.23-15-07.mp4

How is this PR achieving the goal

Hook CNetObjPhysical::CanProcessPendingAttachment and return false if the remote attachments are disabled.

This PR applies to the following area(s)

RedM, Natives

Successfully tested on

Game builds: 1491
Platforms: Windows

Checklist

  • Code compiles and has been tested successfully.
  • Code explains itself well and/or is documented.
  • My commit message explains what the changes do and what they are for.
  • No extra compilation warnings are added by these changes.

Fixes issues

https://discord.com/channels/779705925577080842/779739506286002196/1450523787861819523 CFX Engineering Group

@github-actions github-actions bot added RedM Issues/PRs related to RedM invalid Requires changes before it's considered valid and can be (re)triaged labels Dec 16, 2025
@prikolium-cfx prikolium-cfx added the ready-to-merge This PR is enqueued for merging label Dec 17, 2025
@DaniGP17 DaniGP17 force-pushed the feat/disable-remote-attaches-native branch from 35b452b to bfec4fa Compare December 17, 2025 12:10
@github-actions github-actions bot added triage Needs a preliminary assessment to determine the urgency and required action and removed invalid Requires changes before it's considered valid and can be (re)triaged labels Dec 17, 2025
@DaniGP17
Copy link
Contributor Author

This was disabling all attaches even if you had ownership of the target entity. It's fixed now.

@github-actions github-actions bot added invalid Requires changes before it's considered valid and can be (re)triaged and removed triage Needs a preliminary assessment to determine the urgency and required action labels Dec 17, 2025
@prikolium-cfx prikolium-cfx merged commit 2ad65b6 into citizenfx:master Jan 5, 2026
2 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

invalid Requires changes before it's considered valid and can be (re)triaged ready-to-merge This PR is enqueued for merging RedM Issues/PRs related to RedM

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants