How to make a script that clones a tool from serverstorage to your backpack when you spawn?

Ive come up with this:

function onPlayerSpawned(player)
  if player.Name == "Player1" then
    game.ServerStorage.Tool:Clone().Parent = player.Backpack
  end
end
 
game.Players.PlayerAdded:connect(function(player)
  player.CharacterAdded:connect(function()
    onPlayerSpawned(player)
  end)
end)

Doesnt work? How would I fix?

Found the problem:

if player.Name == "Player1" then

If your testing in Studio, it would be your username. Not Player1 Also use this:

game:GetService("ServerStorage").Tool:Clone().Parent = player.Backpack

Local scripts don't have access to ServerStorage

game.Players.PlayerAdded:connect(function(player)
  player.CharacterAdded:connect(function()
  if player.Name == "yourusername" then
    game.ServerStorage.Tool:Clone().Parent = player.Backpack
  end
  end)
end)