.luacheckrc

@@ -1,7 +0,0 @@
-
-unused_args = false
-allow_defined_top = true
-
-read_globals = {
-	"minetest",
-}

LICENSE

@@ -1,24 +0,0 @@
-BSD 2-Clause License
-
-Copyright (c) 2014-2023, Diego Martínez
-
-Redistribution and use in source and binary forms, with or without
-modification, are permitted provided that the following conditions are met:
-
-1. Redistributions of source code must retain the above copyright notice, this
-   list of conditions and the following disclaimer.
-
-2. Redistributions in binary form must reproduce the above copyright notice,
-   this list of conditions and the following disclaimer in the documentation
-   and/or other materials provided with the distribution.
-
-THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
-AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
-IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
-DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE
-FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
-DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
-SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
-CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
-OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
-OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.

README.md

@@ -1,112 +0,0 @@
-
-# Extended Ban Mod for Minetest
-
-This mod attempts to be an improvement to Minetest's ban system.
-
-* It supports normal bans and temporary bans (from 60 seconds up to the end of
-  time, with 1 second granularity).
-* Records and joins all accounts using the same IP address and several IP
-  addresses using the same name into a single record, and can ban/unban them as
-  a single user.
-* Can ban offline players if you know their IP or username.
-* Holds a record of bans for each user, so moderators and administrators can
-  consult it to know if a player is a repeat offender.
-* Does not modify the default ban database in any way (`ipban.txt').
-* Has an API to ban and check the ban database to allows other mods to manage
-  users (for example, anticheat mods).
-
-## Chat commands
-
-The mod provides the following chat commands. All commands require the `ban`
-privilege.
-
-### `xban`
-
-Bans a player permanently.
-
-**Usage:** `/xban <player_or_ip> <reason>`
-
-**Example:** `/xban 127.0.0.1 Some reason.`
-
-### `xtempban`
-
-Bans a player temporarily.
-
-**Usage:** `/xtempban <player_or_ip> <time> <reason>`
-
-The `time` parameter is a string in the format `<count><unit>` where `<unit>`
-is one of `s` for seconds, `m` for minutes, `h` for hours, `D` for days, `W`
-for weeks, `M` for months, or `Y` for years. If the unit is omitted, it is
-assumed to mean seconds. For example, `42s` means 42 seconds, `1337m` 1337
-minutes, and so on. You can chain more than one such group and they will add
-up. For example, `1Y3M3D7h` will ban for 1 year, 3 months, 3 days and 7 hours.
-
-**Example:** `/xtempban Joe 3600 Some reason.`
-
-### `xunban`
-
-Unbans a player.
-
-**Usage:** `/xunban <player_or_ip>`
-
-**Example:** `/xunban Joe`
-
-### `xban_record`
-
-Shows the ban record on chat.
-
-**Usage:** `/xban_record <player_or_ip>`
-
-This prints one ban entry per line, with the time the ban came into effect,
-the expiration time (if applicable), the reason, and the source of the ban.
-The record is printed to chat with one entry per line.
-
-**Example:** `/xban_record Joe`
-
-### `xban_wl`
-
-Manages the whitelist.
-
-**Usage:** `/xban_wl (add|del|get) <player_or_ip>`
-
-Whitelisted players are allowed on the server even if it's otherwise marked
-as banned. This is useful to only allow certain users from shared computers,
-for example.
-
-The `add` subcommand adds the player to the whitelist. The `del` subcommand
-removes the player from the whitelist. The `get` subcommand checks if the
-player is in the whitelist, and prints the status to chat.
-
-**Example:** `/xban_record add Jane`
-
-### `xban_gui`
-
-Shows a form to consult the database interactively.
-
-**Usage:** `/xban_gui`
-
-## Administrator commands
-
-The following commands require the `server` privilege, so they are only
-available to server administrators.
-
-### `xban_dbi`
-
-Imports ban entries from other database formats.
-
-**Usage:** `/xban_dbi <importer>`
-
-The `importer` argument specifies from which database to import. These are
-the supported import plugins at the time of writing:
-
-* `minetest`: Import entries from Minetest's ban list (`ipban.txt`).
-* `v1`: Old format used by xban (`players.iplist`).
-* `v2`: Old format used by xban (`players.iplist.v2`).
-
-**Example:** `/xban_dbi minetest`
-
-### `xban_cleanup`
-
-Removes all non-banned entries from the xban db.
-
-**Usage:** `/xban_cleanup`

dbimport.lua

@@ -11,13 +11,13 @@ minetest.register_chatcommand("xban_dbi", {
 	privs = { server=true },
 	func = function(name, params)
 		if params == "--list" then
-			local importers = { }
-			for importer in pairs(xban.importers) do
-				table.insert(importers, importer)
+			local names = { }
+			for name in pairs(xban.importers) do
+				table.insert(names, name)
 			end
 			minetest.chat_send_player(name,
 			  ("[xban] Known importers: %s"):format(
-			  table.concat(importers, ", ")))
+			  table.concat(names, ", ")))
 			return
 		elseif not xban.importers[params] then
 			minetest.chat_send_player(name,

gui.lua

@@ -9,7 +9,7 @@ local ESC = minetest.formspec_escape
 local function make_list(filter)
 	filter = filter or ""
 	local list, n, dropped = { }, 0, false
-	for k in minetest.get_auth_handler().iterate() do
+	for k in pairs(minetest.auth_table) do
 		if strfind(k, filter, 1, true) then
 			if n >= MAXLISTSIZE then
 				dropped = true
@@ -61,8 +61,7 @@ local function make_fs(name)
 		"size[16,12]",
 		"label[0,-.1;Filter]",
 		"field[1.5,0;12.8,1;filter;;"..ESC(filter).."]",
-		"field_close_on_enter[filter;false]",
-		"button[14,-.3;2,1;search_submit;Search]",
+		"button[14,-.3;2,1;search;Search]",
 	}
 	local fsn = #fs
 	fsn=fsn+1 fs[fsn] = format("textlist[0,.8;4,9.3;player;%s;%d;0]",
@@ -123,7 +122,7 @@ minetest.register_on_player_receive_fields(function(player, formname, fields)
 		end
 		return
 	end
-	if fields.search_submit or fields.filter then
+	if fields.search then
 		local filter = fields.filter or ""
 		state.filter = filter
 		state.list = make_list(filter)

importers/v2.lua

@@ -8,19 +8,19 @@ function xban.importers.v2()
 		local text = f:read("*a")
 		f:close()
 		local db = minetest.deserialize(text)
-		for _, ent in ipairs(db) do
-			for name in pairs(ent.names) do
+		for _, e in ipairs(db) do
+			for name in pairs(e.names) do
 				local entry = xban.find_entry(name, true)
 				if entry.source ~= "xban:importer_v2" then
 					for nm in pairs(e.names) do
 						entry.names[nm] = true
 					end
-					if ent.banned then
+					if e.banned then
 						entry.banned = true
 						entry.reason = e.banned
 						entry.source = "xban:importer_v2"
-						entry.time = ent.time
-						entry.expires = ent.expires
+						entry.time = e.time
+						entry.expires = e.expires
 						table.insert(entry.record, {
 							source = entry.source,
 							reason = entry.reason,

init.lua

@@ -9,9 +9,9 @@ local tempbans = { }
 local DEF_SAVE_INTERVAL = 300 -- 5 minutes
 local DEF_DB_FILENAME = minetest.get_worldpath().."/xban.db"
 
-local DB_FILENAME = minetest.settings:get("xban.db_filename")
+local DB_FILENAME = minetest.setting_get("xban.db_filename")
 local SAVE_INTERVAL = tonumber(
-  minetest.settings:get("xban.db_save_interval")) or DEF_SAVE_INTERVAL
+  minetest.setting_get("xban.db_save_interval")) or DEF_SAVE_INTERVAL
 
 if (not DB_FILENAME) or (DB_FILENAME == "") then
 	DB_FILENAME = DEF_DB_FILENAME
@@ -24,6 +24,7 @@ local function make_logger(level)
 end
 
 local ACTION = make_logger("action")
+local INFO = make_logger("info")
 local WARNING = make_logger("warning")
 local ERROR = make_logger("error")
 
@@ -41,14 +42,6 @@ local function parse_time(t) --> secs
 	return secs
 end
 
-local function concat_keys(t, sep)
-	local keys = {}
-	for k, _ in pairs(t) do
-		keys[#keys + 1] = k
-	end
-	return table.concat(keys, sep)
-end
-
 function xban.find_entry(player, create) --> entry, index
 	for index, e in ipairs(db) do
 		for name in pairs(e.names) do
@@ -120,7 +113,7 @@ function xban.ban_player(player, source, expires, reason) --> bool, err
 	end
 	ACTION("%s bans %s until %s for reason: %s", source, player,
 	  date, reason)
-	ACTION("Banned Names/IPs: %s", concat_keys(e.names, ", "))
+	ACTION("Banned Names/IPs: %s", table.concat(e.names, ", "))
 	return true
 end
 
@@ -140,7 +133,7 @@ function xban.unban_player(player, source) --> bool, err
 	e.expires = nil
 	e.time = nil
 	ACTION("%s unbans %s", source, player)
-	ACTION("Unbanned Names/IPs: %s", concat_keys(e.names, ", "))
+	ACTION("Unbanned Names/IPs: %s", table.concat(e.names, ", "))
 	return true
 end
 
@@ -306,11 +299,11 @@ minetest.register_chatcommand("xban_wl", {
 		local cmd, plname = params:match("%s*(%S+)%s*(%S+)")
 		if cmd == "add" then
 			xban.add_whitelist(plname, name)
-			ACTION("%s adds %s to whitelist", name, plname)
+			ACTION("%s adds %s to whitelist", source, plname)
 			return true, "Added to whitelist: "..plname
 		elseif cmd == "del" then
 			xban.remove_whitelist(plname)
-			ACTION("%s removes %s to whitelist", name, plname)
+			ACTION("%s removes %s to whitelist", source, plname)
 			return true, "Removed from whitelist: "..plname
 		elseif cmd == "get" then
 			local e = xban.get_whitelist(plname)
@@ -323,7 +316,6 @@ minetest.register_chatcommand("xban_wl", {
 	end,
 })
 
-
 local function check_temp_bans()
 	minetest.after(60, check_temp_bans)
 	local to_rm = { }
@@ -344,12 +336,18 @@ end
 
 local function save_db()
 	minetest.after(SAVE_INTERVAL, save_db)
+	local f, e = io.open(DB_FILENAME, "wt")
 	db.timestamp = os.time()
-	local contents = assert(xban.serialize_db(db))
-	local ok = minetest.safe_file_write(DB_FILENAME, contents)
+	if f then
+		local ok, err = f:write(xban.serialize(db))
 		if not ok then
-		ERROR("Unable to save database")
+			WARNING("Unable to save database: %s", err)
 		end
+	else
+		WARNING("Unable to save database: %s", e)
+	end
+	if f then f:close() end
+	return
 end
 
 local function load_db()
@@ -363,10 +361,10 @@ local function load_db()
 		WARNING("Unable to load database: %s", "Read failed")
 		return
 	end
-	local t, e2 = xban.deserialize_db(cont)
+	local t = minetest.deserialize(cont)
 	if not t then
 		WARNING("Unable to load database: %s",
-		  "Deserialization failed: "..(e2 or "unknown error"))
+		  "Deserialization failed")
 		return
 	end
 	db = t
@@ -378,30 +376,6 @@ local function load_db()
 	end
 end
 
-minetest.register_chatcommand("xban_cleanup", {
-	description = "Removes all non-banned entries from the xban db",
-	privs = { server=true },
-	func = function(name, params)
-		local old_count = #db
-
-		local i = 1
-		while i <= #db do
-			if not db[i].banned then
-				-- not banned, remove from db
-				table.remove(db, i)
-			else
-				-- banned, hold entry back
-				i = i + 1
-			end
-		end
-
-		-- save immediately
-		save_db()
-
-		return true, "Removed " .. (old_count - #db) .. " entries, new db entry-count: " .. #db
-	end,
-})
-
 minetest.register_on_shutdown(save_db)
 minetest.after(SAVE_INTERVAL, save_db)
 load_db()

serialize.lua

@@ -27,44 +27,5 @@ local function my_serialize_2(t, level)
 end
 
 function xban.serialize(t)
-	minetest.log("warning", "[xban2] xban.serialize() is deprecated")
 	return "return {\n"..my_serialize_2(t, 1).."\n}"
 end
-
--- JSON doesn't allow combined string+number keys, this function moves any
--- number keys into an "entries" table
-function xban.serialize_db(t)
-	local res = {}
-	local entries = {}
-	for k, v in pairs(t) do
-		if type(k) == "number" then
-			entries[k] = v
-		else
-			res[k] = v
-		end
-	end
-	res.entries = entries
-	return minetest.write_json(res, true)
-end
-
-function xban.deserialize_db(s)
-	if s:sub(1, 1) ~= "{" then
-		-- Load legacy databases
-		return minetest.deserialize(s)
-	end
-
-	local res, err = minetest.parse_json(s)
-	if not res then
-		return nil, err
-	end
-
-	-- Remove all "null"s added by empty tables
-	for i, entry in ipairs(res.entries or {}) do
-		entry.names = entry.names or {}
-		entry.record = entry.record or {}
-		res[i] = entry
-	end
-	res.entries = nil
-
-	return res
-end