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 AntiCheat.SARDAntiCheat.md #863

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

SARDAntiCheat
Copy link

SteamDB app page links to a few games using this

Brief explanation of the change
Adds SARD as an Anti-Cheat provider for multiplayer games.

@Lifeismana
Copy link
Collaborator

You didn't add a rule to find your ac. Is that an error?
Quickly looking at the 2 games you linked i didn't see a file that would be obviously linked to your ac, are you aware if thefe's one?
Fyi, we don't do manual detection, it's only via regex so if your ac is only packed into the game's exe without having a file for itself, we won't have a way to detect it

@SARDAntiCheat
Copy link
Author

Hello! thanks for feedback, We did the changes.

@SteamDB-Tracker
Copy link
Member

After running changed rules in this pull request (this bot only tests rules.ini changes):

Changed technologies: AntiCheat.SARDAntiCheat

These apps now match: (5) 204940 603850 2202760 2204420 2772750

Please confirm that all these changed apps are correct.

@SARDAntiCheat
Copy link
Author

we are not sure of this one (https://steamdb.info/app/204940/depots/), since Sard AC provides free trial and automated integration. or there can be folder named SARD
so we changed the rule to only check the sardupdater.exe file for now
others definitely has sard ac integrated

@SARDAntiCheat
Copy link
Author

After running changed rules in this pull request (this bot only tests rules.ini changes):

Changed technologies: AntiCheat.SARDAntiCheat

These apps now match: (5) 204940 603850 2202760 2204420 2772750

Please confirm that all these changed apps are correct.

Could you tell us if we need to submit any other information for confirmation?

@SteamDB-Tracker
Copy link
Member

After running changed rules in this pull request (this bot only tests rules.ini changes):

Changed technologies: AntiCheat.SARDAntiCheat

These apps now match: (2) 2202760 2204420

Please confirm that all these changed apps are correct.

@Lifeismana
Copy link
Collaborator

Hey,
main issue is that your updated rule only matches 1 app (Rogue Valley + its playtest) and not the ones you mentioned in your inital PR.
Looking at their game files, nothing clearly pops-up as being from your anticheat so unless you update your "tooling" (?) so that any game that ships with your anti-cheat creates a unique file on build (not on launch) (e.g. license file, but i don't think it makes sense for a proprietary software) there's nothing that can be done (games cannot be manually tagged with a tech)

Also we prefer not to merge detection rules for techs used by a low amount of apps (with some exceptions but 1 detection won't be an exception) so in its current state, this PR wouldn't be merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: To Triage
Development

Successfully merging this pull request may close these issues.

3 participants