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

Add econ to the client #3149

Open
ChillerDragon opened this issue Apr 22, 2022 · 6 comments
Open

Add econ to the client #3149

ChillerDragon opened this issue Apr 22, 2022 · 6 comments

Comments

@ChillerDragon
Copy link
Contributor

Opinions? Concerns?

@ChillerDragon
Copy link
Contributor Author

ChillerDragon commented Apr 22, 2022

Should be doable in a ~5 line diff commit

image

@Robyt3
Copy link
Contributor

Robyt3 commented Apr 22, 2022

Do you mean that the client can connect to servers via econ? Or that the (headless) client can be controlled via econ?

@heinrich5991
Copy link
Contributor

Might open issues wrt. cheating. What programs are allowed to connect to that econ socket in normal gameplay?

@ChillerDragon
Copy link
Contributor Author

Do you mean that the client can connect to servers via econ? Or that the (headless) client can be controlled via econ?

The latter I guess. Adding econ to the client so the client can be controlled using econ. Headless clients would profit from that a lot yea. But also as heinrich said it can be used to upgrade your regular gui client.

Might open issues wrt. cheating. What programs are allowed to connect to that econ socket in normal gameplay?

If one wants to cheat it is much easier to directly modify the client or use a python script to simulate the keyboard.
The only issue I see is it lowers the barrier for quickly building a chat spam bot.

Not saying that it justifies anything but the ddnet client fifo interface has not yet been used maliciously to my knowledge.

@heinrich5991
Copy link
Contributor

If one wants to cheat it is much easier to directly modify the client or use a python script to simulate the keyboard.

Yes, but that's clearly cheating. Usually, the rule is "everything possible with the original client without modifying inputs is not cheating". Adding econ would create a gray zone.

@AssassinTee
Copy link
Contributor

Couldn't you only allow rcon/console to econ clients? I don't see the problem here

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

No branches or pull requests

4 participants