Cheat automatic ban

[kerjinhom (Ker)]

1. What do you want to achieve?
   Speed too fast will be permanently banned

2. What is the issue?
   kicked but not permanently banned
   

StarterPlayer StarterPlayerScript’s localscirpt

local Players = game:GetService("Players")
local LocalPlayer = Players.LocalPlayer
repeat wait() until LocalPlayer.Character
local replicatedstorage = game:GetService("ReplicatedStorage")
local bandatabase = replicatedstorage:WaitForChild("CheatBan")
local localplayeruid = LocalPlayer.UserId

while true do
    wait()
	if LocalPlayer.Character.Humanoid.WalkSpeed >= game.StarterPlayer.CharacterWalkSpeed+1 then
		bandatabase:FireServer(LocalPlayer)
		LocalPlayer:Kick("you are banned")
        wait(10)
    end
end

ServerScriptService’s scirpt

local replicatedstorage = game:GetService("ReplicatedStorage")
local bandatabase = replicatedstorage:WaitForChild("CheatBan")
local datastoreservervice = game:GetService("DataStoreService")
local bandatastore = datastoreservervice:GetDataStore("bandatastore")
local Player = game:GetService("Players")

bandatabase.OnServerEvent:Connect(function()	
	local success, errormessage = pcall(function()
		bandatastore:SetAsync(Player.UserId, true)
	end)
end)

ServerScriptService’s scirpt

local datastoreservervice = game:GetService("DataStoreService")
local bandatastore = datastoreservervice:GetDataStore("bandatastore")

game.Players.PlayerAdded:Connect(function(player)
	local playerUserId = player.UserId

	local banned
	local success, errormessage = pcall(function()
		banned = bandatastore:GetAsync(playerUserId)
	end)
	
	if banned == true then
		player:Kick("(ID:C1)\n\nYou have been banned if this is a bug please report it to the Staff")
	end
end)

[Judgy_Oreo (phoenixwhitefire2000)]

@kerjinhom

local Player = game:GetService("Players")

Rename the variable from Player to Players

@kerjinhom

bandatabase.OnServerEvent:Connect(function()

Put Player inbetween the brackets, like so

bandatabase.OnServerEvent:Connect(function(Player)

@kerjinhom

bandatabase:FireServer(LocalPlayer)

Remove LocalPlayer.

Also, I just want to say, this is very easily bypassable. I suggest you read up on this fantastic post, if you’re looking to make a good anti-cheat:

Why Your Anti-Exploit Sucks

Almost every day the #resources:community-resources section is blessed with an anti-exploit which can’t even do its job because it indulges in some bad practices. Now while these posts are met with valid criticism, and usually get locked within a few hours, I’d like to attempt to educate any aspiring anti-exploit developers with a few of the most common bad practices I see in such posts. My personal big three of the bad practice list are Client-sided security Common false positives Lack of ve…

And this:

Exploiters and how they spoof stuff in your own game

I’ve recently started noticing that people on the devforum are unaware of the true power exploits have and in this tutorial i wanna educate you on how powerful they really are. So let’s start from a recent post i saw about walkspeed. People were doing walkspeed checks by just doing like: if Humanoid.WalkSpeed>16 then LocalPlayer:Kick("Cheater") end Now there are 3 problems with this script. Let’s start with the easiest one: The exploiter could delete the script - this is true but if you hid…

[R3dTuxedo (R3dTux)]

Changing the names of variables might be helpful but I don’t think its the problem. I think its a data store issue. Whats is your output kerjinhom?

[Judgy_Oreo (phoenixwhitefire2000)]

Read the scripts, he’s made a variable called Player which is set to game:GetService("Players"), and then tries to use Player.UserId on that.

[R3dTuxedo (R3dTux)]

Oh yea, my bad. He is treating it like a local script.

[kerjinhom (Ker)]

the problem still exists

local Player = game:GetService("Players")
local LocalPlayer = Player.LocalPlayer
repeat wait() until LocalPlayer.Character
local replicatedstorage = game:GetService("ReplicatedStorage")
local bandatabase = replicatedstorage:WaitForChild("CheatBan")
local localplayeruid = LocalPlayer.UserId

while true do
	wait()
	if LocalPlayer.Character.Humanoid.WalkSpeed >= game.StarterPlayer.CharacterWalkSpeed+1 then
		bandatabase:FireServer()
		LocalPlayer:Kick("you are banned")
		wait(10)
	end
end

I think the problem is here But I tried for hours and still didn’t solve the problem

local replicatedstorage = game:GetService("ReplicatedStorage")
local bandatabase = replicatedstorage:WaitForChild("CheatBan")
local datastoreservervice = game:GetService("DataStoreService")
local bandatastore = datastoreservervice:GetDataStore("bandatastore")
local Player = game:GetService("Players")

bandatabase.OnServerEvent:Connect(function(Player)
	local success, errormessage = pcall(function()
		bandatastore:SetAsync(Player.UserId, true)
	end)
end) 

[Judgy_Oreo (phoenixwhitefire2000)]

@kerjinhom

local Player = game:GetService("Players")

Have you tried renaming Player to Players here like I said? Are there any errors?

[kerjinhom (Ker)]

I have tried but it still doesn’t work

local replicatedstorage = game:GetService("ReplicatedStorage")
local bandatabase = replicatedstorage:WaitForChild("CheatBan")
local datastoreservervice = game:GetService("DataStoreService")
local bandatastore = datastoreservervice:GetDataStore("bandatastore")
local Players = game:GetService("Players")

bandatabase.OnServerEvent:Connect(function(Player)
	local success, errormessage = pcall(function()
		bandatastore:SetAsync(Player.UserId, true)
	end)
end) 

local Players = game:GetService("Players")
local LocalPlayer = Players.LocalPlayer
repeat wait() until LocalPlayer.Character
local replicatedstorage = game:GetService("ReplicatedStorage")
local bandatabase = replicatedstorage:WaitForChild("CheatBan")
local localplayeruid = LocalPlayer.UserId

while true do
	wait()
	if LocalPlayer.Character.Humanoid.WalkSpeed >= game.StarterPlayer.CharacterWalkSpeed+1 then
		bandatabase:FireServer()
		LocalPlayer:Kick("you are banned")
		wait(10)
	end
end

But the only error is this

[Judgy_Oreo (phoenixwhitefire2000)]

Why haven’t you renamed Player to Players in

local Player = game:GetService("Players")

in the image?

[kerjinhom (Ker)]

I’m sorry I took the wrong screenshot but this causes the script to not work

[Judgy_Oreo (phoenixwhitefire2000)]

You need to change

local LocalPlayer = Player.LocalPlayer

to

local LocalPlayer = Players.LocalPlayer

as well

[kerjinhom (Ker)]

it still doesn’t work

local Players = game:GetService("Players")
local LocalPlayer = Players.LocalPlayer
repeat wait() until LocalPlayer.Character
local replicatedstorage = game:GetService("ReplicatedStorage")
local bandatabase = replicatedstorage:WaitForChild("CheatBan")
local localplayeruid = LocalPlayer.UserId

while true do
	wait()
	if LocalPlayer.Character.Humanoid.WalkSpeed >= game.StarterPlayer.CharacterWalkSpeed+1 then
		bandatabase:FireServer()
		LocalPlayer:Kick("you are banned")
		wait(10)
	end
end

I am using RemoteEvent I’ll try to replace it with RemoteFunction later

[Forummer]

--LOCAL

local starterPlayer = game:GetService("StarterPlayer")
local players = game:GetService("Players")
local player = players.LocalPlayer
local character = player.Character or player.CharacterAdded:Wait()
local humanoid = character:WaitForChild("Humanoid")

local replicated = game:GetService("ReplicatedStorage")
local remote = replicated:WaitForChild("RemoteEvent")

humanoid:GetPropertyChangedSignal("WalkSpeed"):Connect(function()
	if humanoid.WalkSpeed > starterPlayer.CharacterWalkSpeed then
		remote:FireServer()
	end
end)

--SERVER

local replicated = game:GetService("ReplicatedStorage")
local remote = replicated.RemoteEvent

remote.OnServerEvent:Connect(function(player)
	player:Kick()
end)

This is working on my end, a single RemoteEvent instance is used and has been placed inside the ReplicatedStorage container.

[9maximuspondal0 (choke)]

Hello! I suggest reading a guide on making a proper anti-exploit as this can be easily bypassed by exploiters.

A Guide to Making Proper Anti-Exploits

Hello, I am Reapimus! Today I am making this guide to the proper way of making an anti-exploit, because far too often on the developer forum I am seeing so many people giving poor anti-exploit advice or trying to make ones that simply won’t work, and I am hoping that by making this guide, more people will learn and know how to make an effective, and proper anti-exploit for their games. So with that introduction out of the way, lets start with this guide! What is an Exploit? To start off with,…

[kerjinhom (Ker)]

good news after a bit of tinkering, I found the root cause was because it was kick too fast can’t upload to the database

[daffyjacks3 (ItzYaBoi_Daffy)]

Well done. I suggest you mark this as solution so people know its been sorted