Skip to content
This repository has been archived by the owner on Nov 16, 2017. It is now read-only.

Restricting rcon output to player #233

Open
wants to merge 2 commits into
base: DDRace
Choose a base branch
from
Open

Restricting rcon output to player #233

wants to merge 2 commits into from

Conversation

east
Copy link

@east east commented May 11, 2013

Since all chat commands use OUTPUT_LEVEL_STANDARD for printing the output gets displayed in every remote console. Without spamprotection this issue can be used to kick authed players out of the server (#232).
Here I added a functionality to restrict output messages to specific players. This workaround might be ugly but the ddrace command system kinda lacks of design

@Tsin
Copy link

Tsin commented May 11, 2013

i see i have access to merge it but ofcourse i wont because i dont feel like i should touch any of this stuff my goal was about issues and thats all i will do so i dont know why i kind of have full access to this git kind of feels strange maybe its because its not possible to lower the access but ok

about this isse it doesnt matter if is ugly or not its a fix that is needed and nobody from current devs will fix it
and this fix is needed also ddrace is planned to be rewritten in near future so it doesnt matter if now some "ugly" code will be added

so guys add it fast ddmax rly needs it

@alandball
Copy link

Doesn't console_output_level control this?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants