Raised This Month: $ Target: $400
 0% 

[CS:GO] Ban By VAC


Post New Thread Reply   
 
Thread Tools Display Modes
Author Message
Visual77
Veteran Member
Join Date: Jan 2009
Old 07-15-2018 , 02:04   Re: [CS:GO] Ban By VAC
Reply With Quote #1

Quote:
Originally Posted by Drixevel View Post
"Very wrong" might be an overreach but I see your point.
he's right about it being "very wrong". strequal should be used when the same word re-appears in different kick messages.

Last edited by Visual77; 07-15-2018 at 02:06.
Visual77 is offline
backwards
AlliedModders Donor
Join Date: Feb 2014
Location: USA
Old 07-15-2018 , 02:49   Re: [CS:GO] Ban By VAC
Reply With Quote #2

Quote:
Originally Posted by Visual77 View Post
he's right about it being "very wrong". strequal should be used when the same word re-appears in different kick messages.
Not exactly, he's still technically correct. it just requires more checks to work as intended. Just combined it with "StrContains" check for not containing "authentication" and other phrases associated with false VAC related disconnection issues.
__________________
I highly recommend joining the SourceMod Discord Server for real time support.
backwards is offline
DarkDeviL
SourceMod Moderator
Join Date: Apr 2012
Old 07-15-2018 , 07:08   Re: [CS:GO] Ban By VAC
Reply With Quote #3

Quote:
Originally Posted by 1337norway View Post
Just combined it with "StrContains" check for not containing "authentication" and other phrases associated with false VAC related disconnection issues.
Sounds very lousy, when you have the option do it "properly" like shown in Bacardi's example in Post #12.

The least you can do when making plugins, is try to do them right.
__________________
Mostly known as "DarkDeviL".

Dropbox FastDL: Public folder will no longer work after March 15, 2017!
For more info, see the [SRCDS Thread], or the [HLDS Thread].
DarkDeviL is offline
backwards
AlliedModders Donor
Join Date: Feb 2014
Location: USA
Old 07-15-2018 , 07:32   Re: [CS:GO] Ban By VAC
Reply With Quote #4

Quote:
Originally Posted by arne1288 View Post
Sounds very lousy, when you have the option do it "properly" like shown in Bacardi's example in Post #12.

The least you can do when making plugins, is try to do them right.
I wouldn't say it's "lousy" as that method Bacardi posted would break easily if the message string was slightly changed in the future. The dynamic way (my suggestion) would be more appropriate for the long term. It's ignorant to believe there's a "proper" way when it comes to something as contrasted as programming.
__________________
I highly recommend joining the SourceMod Discord Server for real time support.
backwards is offline
DarkDeviL
SourceMod Moderator
Join Date: Apr 2012
Old 07-15-2018 , 07:42   Re: [CS:GO] Ban By VAC
Reply With Quote #5

Quote:
Originally Posted by 1337norway View Post
I wouldn't say it's "lousy" as that method Bacardi posted would break easily if the message string was slightly changed in the future. The dynamic way (my suggestion) would be more appropriate for the long term. It's ignorant to believe there's a "proper" way when it comes to something as contrasted as programming.
Cutting corners and claiming to be "future proof", especially when you never know what happens in the future, and when it will lead to false positives; that is clearly "lousy".

The ignorant way, to take your word, would be check multiple StrEqual/StrContains, when you have the option right now to do it right and use one single, that wouldn't cause a single false positive.

You have some options and things available right now, those are the only things you can depend on at the time being. If things change in the future, you should adjust things at that time in the future.

A "Ban By VAC", to quote the title, is related to when a user is VAC banned, and NOT when there are anything else related to the VAC system, that isn't the exact ban.
__________________
Mostly known as "DarkDeviL".

Dropbox FastDL: Public folder will no longer work after March 15, 2017!
For more info, see the [SRCDS Thread], or the [HLDS Thread].

Last edited by DarkDeviL; 07-15-2018 at 07:46.
DarkDeviL is offline
backwards
AlliedModders Donor
Join Date: Feb 2014
Location: USA
Old 07-15-2018 , 07:57   Re: [CS:GO] Ban By VAC
Reply With Quote #6

Quote:
Originally Posted by arne1288 View Post
Cutting corners and attempting to be "future proof", especially when you never know what happens in the future, and when it may lead to false positives; that is clearly "lousy".

The ignorant way, to take your word, would be check multiple StrEqual/StrContains, when you have the option right now to do it right and use one single, that wouldn't cause a single false positive.

You have some options and things available right now, those are the only things you can depend on at the time being. If things change in the future, you should adjust things at that time in the future.

A "Ban By VAC", to quote the title, is related to when a user is VAC banned, and NOT when there are anything else related to the VAC system, that isn't the exact ban.
"cutting corners" implies that short cuts are taken to reduce the complexity of code in this situation. Obviously adding more code for separate checks would be the adverse of this. You are right, you never know what will happen in the future with updates. You don't know if they will change the string or introduce new strings which would make both scenarios equal. You'd have to come back and manipulate the program to account for these changes regardless. Adding multiple string checks wouldn't correlate to increased performance overhead in this case as the single longer string check would be even worse. You've made no valid logical statements for why one would be preferred over the other. It's clear you are upset over something here but attacking users on a public forum as an active moderator over disagreeing about programming choices is a bit ridiculous
__________________
I highly recommend joining the SourceMod Discord Server for real time support.
backwards is offline
DarkDeviL
SourceMod Moderator
Join Date: Apr 2012
Old 07-15-2018 , 08:22   Re: [CS:GO] Ban By VAC
Reply With Quote #7

Quote:
Originally Posted by 1337norway View Post
"cutting corners" implies that short cuts are taken to reduce the complexity of code in this situation. Obviously adding more code for separate checks would be the adverse of this. You are right, you never know what will happen in the future with updates. You don't know if they will change the string or introduce new strings which would make both scenarios equal. You'd have to come back and manipulate the program to account for these changes regardless. Adding multiple string checks wouldn't correlate to increased performance overhead in this case as the single longer string check would be even worse. You've made no valid logical statements for why one would be preferred over the other. It's clear you are upset over something here but attacking users on a public forum as an active moderator over disagreeing about programming choices is a bit ridiculous
And how come that cutting most of the phase away, before the check, isn't a "short cut" to you?

Kigen's anti cheat were failing after some Valve update in the past, causing false positives and thereby false bans. The lousy administrator of one of the servers where a such thing was happening, and were every single client was being banned, was once unwilling to "delete" the bans completely and remove the "ban history" .

In short words, it would be a matter of minutes to a couple of hours usually, before Google would know about the bans and list your Steam ID as being a previous offender by an anti cheat.

If, as you say, you do it with checking if "VAC" is in the phrase, and they will change it in the future, you will get innocent people banned from your way of doing it.

Using the full phase allows obviously requires future maintenance if that day will ever come, but is a simple phrase change, that will not cause any false positives.

- How would you personally feel, if your Steam profile had incorrect histories (that be, at third party sites), regarding VAC bans that you have never ever had? People may actually act on this!

If there is a reason for the ban, sure, go ahead, ...

Hitting innocents is NOT something I would be proud of, and while I have seen at least 3-4 VAC-related phrases, with only one of them being related to the exact ban, you're definitely going to end up hitting innocent people with your way of doing it.


There is, unfortunately, no ways to distinguish between "personal opinions" and "moderator opinions" form a moderator on any forums out there:

You're clearly misunderstanding my intentions with these messages, as they have never (neither as personal, nor as a moderator) been meant to be "attacking" you (or anyone else), but simply to help any people to make the code right.
__________________
Mostly known as "DarkDeviL".

Dropbox FastDL: Public folder will no longer work after March 15, 2017!
For more info, see the [SRCDS Thread], or the [HLDS Thread].

Last edited by DarkDeviL; 07-15-2018 at 08:25.
DarkDeviL is offline
Visual77
Veteran Member
Join Date: Jan 2009
Old 07-15-2018 , 07:49   Re: [CS:GO] Ban By VAC
Reply With Quote #8

Quote:
Originally Posted by 1337norway View Post
I wouldn't say it's "lousy" as that method Bacardi posted would break easily if the message string was slightly changed in the future. The dynamic way (my suggestion) would be more appropriate for the long term. It's ignorant to believe there's a "proper" way when it comes to something as contrasted as programming.
it is lousy. valve never changes those kick messages. prolly been the same since vac was introduced i guess

Last edited by Visual77; 07-15-2018 at 07:53.
Visual77 is offline
Reply



Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump


All times are GMT -4. The time now is 22:09.


Powered by vBulletin®
Copyright ©2000 - 2024, vBulletin Solutions, Inc.
Theme made by Freecode