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

Prevent server from sending "bind" command #267

Closed
xPaw opened this issue Feb 17, 2013 · 13 comments
Closed

Prevent server from sending "bind" command #267

xPaw opened this issue Feb 17, 2013 · 13 comments

Comments

@xPaw
Copy link

xPaw commented Feb 17, 2013

No description provided.

@MPNumB
Copy link

MPNumB commented Feb 17, 2013

I agree. And special char check before and after the command would be nice as well, to avoid bypass methods.

P.S. Don't forget "unbind" and "unbindall".

@ghost
Copy link

ghost commented Feb 18, 2013

I don't get it why people still changing their config.cfg. Why they are not creating a new file called myshittyconfig.cfg and execute once the cfg with "exec myshittyconfig,cfg".

I agree that such servers are shit, but it is also useful if you ask and inform users about your changes. Black sheeps are always out there, and if they want to change soemthing they can do that without the bind command. Keep it as it is, and set the server on your blacklist.

@ghost ghost assigned alfred-valve Feb 18, 2013
@ghost
Copy link

ghost commented Feb 20, 2013

Also the bind command is useful, e.g. to bind a mod special function to a key. Especially for non-geek users who doesn't have any idea how to do it. Ofcourse if they have accepted it or executed it by themself e.g. through the chat system.

Player A: types "setbind" into the chat
Server: executes the right commands on Player A
Player A: is happy, because he couldn't do that by himself

Instead of blocking bind I would suggest to add a feature/function like reset_settings or load_defaults, so all variables turning back to their original (e.g. from config_defaults.cfg). Also I hope you guys are using the userconfig.cfg to edit your settings. This files is save even without "read only" attribute. Or better you are using a custom named cfg.

I never had an issue with using userconfig.cfg or [insert name here].cfg. I also never had "read only" on config.cfg.

@SparkiTP
Copy link

SparkiTP commented Mar 7, 2013

Completetly agree with @leon291.

Please allow players to recieve bind commands from server.
Atleast, provide a cvar, so that client can decide to allow those commands or not.
Thanks!

Awaiting your reply @alfred-valve

@ghost
Copy link

ghost commented Mar 7, 2013

@SparkiTP the cvar is provided - cl_filterstuffcmd

@SparkiTP
Copy link

SparkiTP commented Mar 7, 2013

Oh ok, thanks for that information. @Maxkorz
Can that commands be changed only by client or also from server side?
Will check that command, meanwhile.

@ghost
Copy link

ghost commented Mar 7, 2013

This cvar can be changed only by client. Even client_cmd( player, "cl_filterstuffcmd 1"); won't work:

] cl_filterstuffcmd 
"cl_filterstuffcmd" is "0"
] amx_slowhack Neo
Server tried to send invalid command:"cl_filterstuffcmd 0
"Server tried to send invalid command:"cl_filterstuffcmd 1
"Server tried to send invalid command:"cl_filterstuffcmd 0

@SparkiTP
Copy link

SparkiTP commented Mar 7, 2013

Can someone explain what commands will be blocked with setting cl_filterstuffcmd to 0/1?

https://gist.github.com/SparkiTP/5108100

@ghost
Copy link

ghost commented Mar 7, 2013

Sure.

_restart
_set_vid_level
_setaddons_folder
_setgamedir
_sethdmodels
_setrenderer
_setvideomode
gl_*
r_*
hud_*
cl_*
m_*
rate
connect
retry
timerefresh
alias
bind
unbind
unbindall
cd
exec
exit
kill
quit
say
setinfo
sensitivity
sys_ticrate
writecfg
removedemo
ex_interp
developer
fps_max
speak_enabled
voice_enable
volume
mp3volume
motd_write

If cl_filterstuffcmd is set to 0:

motd_write
cl_filterstuffcmd

I could miss something.

@SparkiTP
Copy link

SparkiTP commented Mar 7, 2013

But, in my case, setting this cl_filterstuffcmd to 0 / 1, does not allow client CS to recieve bind commands :(
Please refer to the gist link, I gave in last post.

@ghost
Copy link

ghost commented Mar 7, 2013

Then create a new issue and wait for @alfred-valve to reply. I don't think it should be blocked when cl_filterstuffcmd 0. At least it wasn't.

@SparkiTP
Copy link

SparkiTP commented Mar 7, 2013

Thanks for your suggestion.
Ok, does the same bug, appear for you?
Can you check?

New issue created: #733

@ghost
Copy link

ghost commented Mar 7, 2013

of course I checked it before suggesting you to create a new issue, so yes, I confirm it :)

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

No branches or pull requests

4 participants