sztest/szutilpack-cd2025
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Handle authed but not joined players

Browse Source 
Further tighten the race condition gap between
players authing and finishing joining.  The
meaning of last_login is apparently officially the
last join time, but if the last successful auth time
was later, use that as the known account age.
This commit is contained in:
Aaron Suen 2022-11-05 13:36:27 -04:00
parent 8d607b360c
commit 1a2d41b0b3
1 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
szutil_pruneplayers/init.lua
View File

@ -60,6 +60,12 @@ end
------------------------------------------------------------------------
local lastauth = {}
minetest.register_on_authplayer(function(name, _, is_success)
if not is_success then return end
lastauth[name] = os_time()
end)
local function check(pname)
if protectnames[pname] or minetest.get_player_by_name(pname) then return end
@ -69,10 +75,14 @@ local function check(pname)
if data.privileges[noprune] then return end
local seen = data.last_login
local authtime = lastauth[pname]
if authtime and authtime > seen then seen = authtime end
local now = os_time()
local daysago = (now - data.last_login) / 86400
local daysago = (now - seen) / 86400
if daysago <= maxdays then return end
lastauth[pname] = nil
minetest.log("warning", string_format(
"Deleting player %q last seen %0.1f day(s) ago",
pname, daysago))