Security improvements and DoS protection for client indicator server

[ashboundscar]

No description provided.

[hylooz]

Зачем вы создали отдельный коммит для изменения созданной вами ранее переменной? Пожалуйста, используйте git commit --amend --no-edit.

[hylooz]

Почему вы выбрали именно значение 4 в качестве максимального? В DDNet существуют не только стандартные серверы DDRace, где это ограничение равно четырём.

[ashboundscar]

it is just a default setting that can be changed based on specific server needs, it was set to 4 because i don't expect people to launch more than 2 client instances. back to 16 if you're so critic about it

[hylooz]

back to 16 if you're so critic about it

Моя главная мысль была в том, чтобы это не сломало логику для нестандартных или кастомных серверов.

Так как это отдельный сервис на Rust, мы, очевидно, не можем просто взять и использовать C++ константу MAX_CLIENTS_PER_IP из server.cpp. Но если предполагается, что владельцы серверов могут менять этот лимит под свои нужды, то, возможно, это вообще не должно быть жестко зашитой const на этапе компиляции?

Может, имеет смысл вынести это в аргументы командной строки (например, --max-entries-per-ip)? Тогда администраторы смогут настраивать лимит без необходимости перекомпилировать исходный код.

[ashboundscar]

back to 16 if you're so critic about it

Моя главная мысль была в том, чтобы это не сломало логику для нестандартных или кастомных серверов.

Так как это отдельный сервис на Rust, мы, очевидно, не можем просто взять и использовать C++ константу MAX_CLIENTS_PER_IP из server.cpp. Но если предполагается, что владельцы серверов могут менять этот лимит под свои нужды, то, возможно, это вообще не должно быть жестко зашитой const на этапе компиляции?

Может, имеет смысл вынести это в аргументы командной строки (например, --max-entries-per-ip)? Тогда администраторы смогут настраивать лимит без необходимости перекомпилировать исходный код.

you're confusing this rust server with the game server itself. ddnet limits are irrelevant because this is a private indicator backend for bestclient, not a public game server. adding cli arguments for a single constant is just unnecessary customization bloat for IRC service. if the developer wants to change a limit, they change one line of code and rebuild in a second. stop giving stupid ai advice without looking at the problem. and use english please, it's the repository standard.