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

FiveM F8 quit message. #2364

Closed
itsAdminPlus opened this issue Jan 29, 2024 · 5 comments
Closed

FiveM F8 quit message. #2364

itsAdminPlus opened this issue Jan 29, 2024 · 5 comments
Labels
enhancement Feature or other request that adds functionality or improved usability

Comments

@itsAdminPlus
Copy link

What happened?

Players fake crashing their game by clicking F8 & putting a message after quit.

Not sure if there's a possible fix you guys can possibly implement here, but depending the message they insert it is not noticeable to know which ones are fake/real unless they have a typo in the message or what-not, as we sometimes look when people quit mid roleplay scene or actually crash mid roleplay scene.

Expected result

Possibly just state "disconnected" or just dont display any args after quit

Reproduction steps

  1. click F8 when ingame
  2. type "quit"
  3. put a message after quit like this: quit "Game crashed: GTA5_b2699.exe!sub_1411E3C8C (0x3f))"
    02f0cc793eb0a1a563f7e7139aa593d7
    2

Importancy

Security issue

Area(s)

FiveM, FXServer

Specific version(s)

7329

Additional information

No response

@itsAdminPlus itsAdminPlus added bug triage Needs a preliminary assessment to determine the urgency and required action labels Jan 29, 2024
@slashkeyvalue
Copy link
Contributor

#1424 (comment)

@aqua3man
Copy link

#1424 (comment) Answer is sound. Though some convar that would allow to disable user provided quit messages would be nice to have. It would limit the scope of this kind of abuse.

@Disquse
Copy link
Contributor

Disquse commented Jan 31, 2024

@aqua3man in this scenario you won't be able to see any quit reason at all. As was said, all quit reasons are "user provided", coming from a client and can't be trusted.

@aqua3man
Copy link

aqua3man commented Feb 1, 2024

@aqua3man in this scenario you won't be able to see any quit reason at all. As was said, all quit reasons are "user provided", coming from a client and can't be trusted.

Yes I understand. What I am talking about is limiting the affect. When every user can /quit , it has same effect as when every user can load client side scripts. Eveyone knows that it can be done, but when it is limited by client, the scope of abuse is also limited. What I am suggesting is limiting parameter . I understand that scripts can be loaded, cheats can be used, client can be abused. The purpose is not to fight with cheaters. Cheaters is another scope and another topic. If every user couldn't provide the message it could be better "compass" for server admin.

@blattersturm
Copy link
Contributor

blattersturm commented Feb 1, 2024

Quit reasons are barely even 'abuse' and this is in no way comparable to 'loading client side scripts'. People can quit in any way imaginable (e.g. intentionally corrupting memory or disconnecting their internet connection), and crash messages are only provided as a courtesy for diagnostics.

If players have an incentive to lie about quitting, you have a people problem, not a technical problem.

A possible suggestion could be prefixing user-provided quit reasons with 'Quit: ' like IRC clients used to do as well.

@gottfriedleibniz gottfriedleibniz added enhancement Feature or other request that adds functionality or improved usability and removed bug triage Needs a preliminary assessment to determine the urgency and required action labels Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Feature or other request that adds functionality or improved usability
Projects
None yet
Development

No branches or pull requests

6 participants