From b64e360b26fcea45faaf4caf8d65db204efdd450 Mon Sep 17 00:00:00 2001
From: metamuffin <metamuffin@disroot.org>
Date: Mon, 23 Dec 2024 20:01:25 +0100
Subject: username length limit

---
 server/src/main.rs   | 2 +-
 server/src/server.rs | 8 ++++++--
 2 files changed, 7 insertions(+), 3 deletions(-)

(limited to 'server')

diff --git a/server/src/main.rs b/server/src/main.rs
index 2d5ba390..7f3de073 100644
--- a/server/src/main.rs
+++ b/server/src/main.rs
@@ -248,7 +248,7 @@ async fn run(args: Args) -> anyhow::Result<()> {
             info!("{id:?} connected");
             while let Some(Ok(message)) = read.next().await {
                 let packet = match message {
-                    Message::Text(line) => match serde_json::from_str(&line) {
+                    Message::Text(line) if line.len() < 8196 => match serde_json::from_str(&line) {
                         Ok(p) => p,
                         Err(e) => {
                             warn!("Invalid json packet: {e}");
diff --git a/server/src/server.rs b/server/src/server.rs
index 9928ac90..462e95d4 100644
--- a/server/src/server.rs
+++ b/server/src/server.rs
@@ -369,8 +369,12 @@ impl Server {
                 id,
                 class,
             } => {
-                if name.len() > 64 {
-                    return Err(tre!("s.error.username_length_limit"));
+                if name.chars().count() > 32 || name.len() > 64 {
+                    return Err(tre!(
+                        "s.error.username_length_limit",
+                        s = "32".to_string(),
+                        s = "64".to_string()
+                    ));
                 }
                 if self.game.players.len() > 64 {
                     return Err(tre!("s.error.too_many_players"));
-- 
cgit v1.2.3-70-g09d2