From 7217499a9be14ebe81dd2c912fe50e2bb1843f1c Mon Sep 17 00:00:00 2001 From: Legendaxe <102746941+Legendaxe@users.noreply.github.com> Date: Fri, 19 Apr 2024 21:31:35 +0300 Subject: [PATCH 01/10] whitelist auto update --- modular_ss220/whitelist/code/whitelist.dm | 27 +++++++++++++++++------ 1 file changed, 20 insertions(+), 7 deletions(-) diff --git a/modular_ss220/whitelist/code/whitelist.dm b/modular_ss220/whitelist/code/whitelist.dm index b51b5c26067b..890e32d99153 100644 --- a/modular_ss220/whitelist/code/whitelist.dm +++ b/modular_ss220/whitelist/code/whitelist.dm @@ -49,14 +49,27 @@ qdel(whitelist_query) -/mob/new_player/Login() - if(!(ckey in GLOB.configuration.overflow.overflow_whitelist)) - check_whitelist() - . = ..() - /world/IsBanned(key, address, computer_id, type, check_ipintel, check_2fa, check_guest, log_info, check_tos) var/ckey = ckey(key) - if(GLOB.configuration.overflow.reroute_cap == 0.5 && ckey && !(ckey in GLOB.configuration.overflow.overflow_whitelist)) - return list("reason"="no-whitelist", "desc"="\nПричина: Вас ([key]) нет в вайтлисте этого сервера. Приобрести доступ возможно у одного из стримеров Банды за баллы канала или записаться самостоятельно с помощью команды в дискорде, доступной сабам бусти, начиная со второго тира.") + + if(GLOB.configuration.overflow.reroute_cap == 0.5) + var/list/denied_entry = list("reason"="no-whitelist", "desc"="\nПричина: Вас ([key]) нет в вайтлисте этого сервера. Приобрести доступ возможно у одного из стримеров Банды за баллы канала или записаться самостоятельно с помощью команды в дискорде, доступной сабам бусти, начиная со второго тира.") + var/datum/db_query/whitelist_query = SSdbcore.NewQuery({" + SELECT ckey FROM ckey_whitelist WHERE ckey=:ckey AND + is_valid=1 AND port=:port AND date_start<=NOW() AND + (NOW() Date: Fri, 19 Apr 2024 21:35:35 +0300 Subject: [PATCH 02/10] redundant proc --- modular_ss220/whitelist/code/whitelist.dm | 19 ------------------- 1 file changed, 19 deletions(-) diff --git a/modular_ss220/whitelist/code/whitelist.dm b/modular_ss220/whitelist/code/whitelist.dm index 890e32d99153..171420bf2696 100644 --- a/modular_ss220/whitelist/code/whitelist.dm +++ b/modular_ss220/whitelist/code/whitelist.dm @@ -30,25 +30,6 @@ load_whitelist() -/mob/new_player/proc/check_whitelist() - if(!GLOB.configuration.overflow.reroute_cap || !SSdbcore.IsConnected()) - return - var/datum/db_query/whitelist_query = SSdbcore.NewQuery({" - SELECT ckey FROM ckey_whitelist WHERE ckey=:ckey AND - is_valid=1 AND port=:port AND date_start<=NOW() AND - (NOW() Date: Fri, 19 Apr 2024 21:40:14 +0300 Subject: [PATCH 03/10] redundant verb --- modular_ss220/whitelist/code/whitelist.dm | 9 --------- 1 file changed, 9 deletions(-) diff --git a/modular_ss220/whitelist/code/whitelist.dm b/modular_ss220/whitelist/code/whitelist.dm index 171420bf2696..6b1de87c4ecf 100644 --- a/modular_ss220/whitelist/code/whitelist.dm +++ b/modular_ss220/whitelist/code/whitelist.dm @@ -21,15 +21,6 @@ qdel(whitelist_query) -/client/proc/update_whitelist() - set name = "Update whitelist" - set category = "Server" - - if(!check_rights(R_SERVER)) - return - - load_whitelist() - /world/IsBanned(key, address, computer_id, type, check_ipintel, check_2fa, check_guest, log_info, check_tos) var/ckey = ckey(key) From ef8ba4dc34615970644bf1231f27b4bb6bfa991b Mon Sep 17 00:00:00 2001 From: Legendaxe <102746941+Legendaxe@users.noreply.github.com> Date: Fri, 19 Apr 2024 21:42:40 +0300 Subject: [PATCH 04/10] linter_fix --- modular_ss220/whitelist/_whitelist.dm | 1 - 1 file changed, 1 deletion(-) diff --git a/modular_ss220/whitelist/_whitelist.dm b/modular_ss220/whitelist/_whitelist.dm index 3ed35899a634..2baa73d34ce4 100644 --- a/modular_ss220/whitelist/_whitelist.dm +++ b/modular_ss220/whitelist/_whitelist.dm @@ -5,4 +5,3 @@ /datum/modpack/whitelist/initialize() load_whitelist() - GLOB.admin_verbs_server |= /client/proc/update_whitelist From bd182bba244e314f53c3076ac6ea9bce762bed84 Mon Sep 17 00:00:00 2001 From: Legendaxe <102746941+Legendaxe@users.noreply.github.com> Date: Fri, 19 Apr 2024 21:50:54 +0300 Subject: [PATCH 05/10] gaxeer review --- modular_ss220/whitelist/code/whitelist.dm | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/modular_ss220/whitelist/code/whitelist.dm b/modular_ss220/whitelist/code/whitelist.dm index 6b1de87c4ecf..3338122c7f15 100644 --- a/modular_ss220/whitelist/code/whitelist.dm +++ b/modular_ss220/whitelist/code/whitelist.dm @@ -24,7 +24,7 @@ /world/IsBanned(key, address, computer_id, type, check_ipintel, check_2fa, check_guest, log_info, check_tos) var/ckey = ckey(key) - if(GLOB.configuration.overflow.reroute_cap == 0.5) + if(GLOB.configuration.overflow.reroute_cap == 0.5 && сkey && !(ckey in GLOB.configuration.overflow.overflow_whitelist)) var/list/denied_entry = list("reason"="no-whitelist", "desc"="\nПричина: Вас ([key]) нет в вайтлисте этого сервера. Приобрести доступ возможно у одного из стримеров Банды за баллы канала или записаться самостоятельно с помощью команды в дискорде, доступной сабам бусти, начиная со второго тира.") var/datum/db_query/whitelist_query = SSdbcore.NewQuery({" SELECT ckey FROM ckey_whitelist WHERE ckey=:ckey AND @@ -32,16 +32,16 @@ (NOW() Date: Fri, 19 Apr 2024 21:52:28 +0300 Subject: [PATCH 06/10] =?UTF-8?q?delete=20=D0=B2=D0=B8=20query?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- modular_ss220/whitelist/code/whitelist.dm | 1 + 1 file changed, 1 insertion(+) diff --git a/modular_ss220/whitelist/code/whitelist.dm b/modular_ss220/whitelist/code/whitelist.dm index 3338122c7f15..8083cfc0c7c5 100644 --- a/modular_ss220/whitelist/code/whitelist.dm +++ b/modular_ss220/whitelist/code/whitelist.dm @@ -39,6 +39,7 @@ while(whitelist_query.NextRow()) var/ckey_from_db = whitelist_query.item[1] GLOB.configuration.overflow.overflow_whitelist |= ckey_from_db + qdel(whitelist_query) return ..() qdel(whitelist_query) From 3c2b0d1a6fbbaabc465800803769726264b709bd Mon Sep 17 00:00:00 2001 From: Legendaxe <102746941+Legendaxe@users.noreply.github.com> Date: Fri, 19 Apr 2024 21:57:28 +0300 Subject: [PATCH 07/10] kill nested if --- modular_ss220/whitelist/code/whitelist.dm | 35 +++++++++++------------ 1 file changed, 17 insertions(+), 18 deletions(-) diff --git a/modular_ss220/whitelist/code/whitelist.dm b/modular_ss220/whitelist/code/whitelist.dm index 8083cfc0c7c5..457c12fa6b4f 100644 --- a/modular_ss220/whitelist/code/whitelist.dm +++ b/modular_ss220/whitelist/code/whitelist.dm @@ -23,26 +23,25 @@ /world/IsBanned(key, address, computer_id, type, check_ipintel, check_2fa, check_guest, log_info, check_tos) var/ckey = ckey(key) + if(GLOB.configuration.overflow.reroute_cap != 0.5 || !сkey || ckey in GLOB.configuration.overflow.overflow_whitelist) + return ..() - if(GLOB.configuration.overflow.reroute_cap == 0.5 && сkey && !(ckey in GLOB.configuration.overflow.overflow_whitelist)) - var/list/denied_entry = list("reason"="no-whitelist", "desc"="\nПричина: Вас ([key]) нет в вайтлисте этого сервера. Приобрести доступ возможно у одного из стримеров Банды за баллы канала или записаться самостоятельно с помощью команды в дискорде, доступной сабам бусти, начиная со второго тира.") - var/datum/db_query/whitelist_query = SSdbcore.NewQuery({" - SELECT ckey FROM ckey_whitelist WHERE ckey=:ckey AND - is_valid=1 AND port=:port AND date_start<=NOW() AND - (NOW() Date: Fri, 19 Apr 2024 21:58:49 +0300 Subject: [PATCH 08/10] cyrillic C moment --- modular_ss220/whitelist/code/whitelist.dm | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/modular_ss220/whitelist/code/whitelist.dm b/modular_ss220/whitelist/code/whitelist.dm index 457c12fa6b4f..caa9ce6b05c5 100644 --- a/modular_ss220/whitelist/code/whitelist.dm +++ b/modular_ss220/whitelist/code/whitelist.dm @@ -23,8 +23,9 @@ /world/IsBanned(key, address, computer_id, type, check_ipintel, check_2fa, check_guest, log_info, check_tos) var/ckey = ckey(key) - if(GLOB.configuration.overflow.reroute_cap != 0.5 || !сkey || ckey in GLOB.configuration.overflow.overflow_whitelist) - return ..() + + if(GLOB.configuration.overflow.reroute_cap != 0.5 || !ckey || ckey in GLOB.configuration.overflow.overflow_whitelist) + return ..() var/list/denied_entry = list("reason"="no-whitelist", "desc"="\nПричина: Вас ([key]) нет в вайтлисте этого сервера. Приобрести доступ возможно у одного из стримеров Банды за баллы канала или записаться самостоятельно с помощью команды в дискорде, доступной сабам бусти, начиная со второго тира.") var/datum/db_query/whitelist_query = SSdbcore.NewQuery({" From d03819ed38f98364ce171a5f24b538c0f57c14ad Mon Sep 17 00:00:00 2001 From: Legendaxe <102746941+Legendaxe@users.noreply.github.com> Date: Fri, 19 Apr 2024 22:01:57 +0300 Subject: [PATCH 09/10] linter fix --- modular_ss220/whitelist/code/whitelist.dm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/modular_ss220/whitelist/code/whitelist.dm b/modular_ss220/whitelist/code/whitelist.dm index caa9ce6b05c5..2ba593602bef 100644 --- a/modular_ss220/whitelist/code/whitelist.dm +++ b/modular_ss220/whitelist/code/whitelist.dm @@ -24,7 +24,7 @@ /world/IsBanned(key, address, computer_id, type, check_ipintel, check_2fa, check_guest, log_info, check_tos) var/ckey = ckey(key) - if(GLOB.configuration.overflow.reroute_cap != 0.5 || !ckey || ckey in GLOB.configuration.overflow.overflow_whitelist) + if(GLOB.configuration.overflow.reroute_cap != 0.5 || !ckey || ckey in GLOB.configuration.overflow.overflow_whitelist) return ..() var/list/denied_entry = list("reason"="no-whitelist", "desc"="\nПричина: Вас ([key]) нет в вайтлисте этого сервера. Приобрести доступ возможно у одного из стримеров Банды за баллы канала или записаться самостоятельно с помощью команды в дискорде, доступной сабам бусти, начиная со второго тира.") From 2db15b30a2e9a7691c03b868bea4b11813cb9328 Mon Sep 17 00:00:00 2001 From: Legendaxe <102746941+Legendaxe@users.noreply.github.com> Date: Fri, 19 Apr 2024 22:05:52 +0300 Subject: [PATCH 10/10] linter fix2 --- modular_ss220/whitelist/code/whitelist.dm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/modular_ss220/whitelist/code/whitelist.dm b/modular_ss220/whitelist/code/whitelist.dm index 2ba593602bef..60a23c6a9b6a 100644 --- a/modular_ss220/whitelist/code/whitelist.dm +++ b/modular_ss220/whitelist/code/whitelist.dm @@ -24,7 +24,7 @@ /world/IsBanned(key, address, computer_id, type, check_ipintel, check_2fa, check_guest, log_info, check_tos) var/ckey = ckey(key) - if(GLOB.configuration.overflow.reroute_cap != 0.5 || !ckey || ckey in GLOB.configuration.overflow.overflow_whitelist) + if(GLOB.configuration.overflow.reroute_cap != 0.5 || !ckey || (ckey in GLOB.configuration.overflow.overflow_whitelist)) return ..() var/list/denied_entry = list("reason"="no-whitelist", "desc"="\nПричина: Вас ([key]) нет в вайтлисте этого сервера. Приобрести доступ возможно у одного из стримеров Банды за баллы канала или записаться самостоятельно с помощью команды в дискорде, доступной сабам бусти, начиная со второго тира.")