adam/cards
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

limit accepted text input length

Browse source
This commit is contained in:
Adam 2024-10-07 21:03:03 -04:00
parent d490086793
commit 7cb2d14c01
3 changed files with 7 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
server/src/game_handler.rs
View file

@ -330,7 +330,7 @@ impl GameHandler {
if let Some(host) = self.state.online_users.read().unwrap().get(&addr) { if let Some(host) = self.state.online_users.read().unwrap().get(&addr) {
// Create manifest // Create manifest
let manifest = NewGameManifest { let manifest = NewGameManifest {
name: new_game.name, name: new_game.name[..64].to_string(), // Limit name length to 64
host: host.clone(), host: host.clone(),
packs: new_game packs: new_game
.packs .packs

2
server/src/message_handler.rs
View file

@ -26,7 +26,7 @@ impl MessageHandler {
Message::Text(text) => match text { Message::Text(text) => match text {
_chat_message if let Ok(chat_message) = from_str::<ChatMessage>(&text) => { _chat_message if let Ok(chat_message) = from_str::<ChatMessage>(&text) => {
// TODO: This should be delegated to user handler and an outgoing message and/or chat handler // TODO: This should be delegated to user handler and an outgoing message and/or chat handler
let msg = format! {"{0}: {1}", self.state.online_users.read().unwrap().get(&addr).unwrap().read().unwrap().name, chat_message.text}; let msg = format! {"{0}: {1}", self.state.online_users.read().unwrap().get(&addr).unwrap().read().unwrap().name, chat_message.text[..1024].to_string()};
tracing::debug!("{msg}"); tracing::debug!("{msg}");
self.state self.state
.broadcast_tx .broadcast_tx

7
server/src/user_handler.rs
View file

@ -140,8 +140,9 @@ impl UserHandler {
/// Handle user login /// Handle user login
async fn login(&self, request: UserLogInRequest, addr: SocketAddr) { async fn login(&self, request: UserLogInRequest, addr: SocketAddr) {
let username_max_len = 66; // This is the longest name the generator may produce right now
let broadcast_tx = self.state.broadcast_tx.clone(); let broadcast_tx = self.state.broadcast_tx.clone();
let new_name = request.username.clone(); let new_name = request.username[..username_max_len].to_string();
let old_name; let old_name;
if let Some(user) = self.state.online_users.read().unwrap().get(&addr) { if let Some(user) = self.state.online_users.read().unwrap().get(&addr) {
@ -221,7 +222,9 @@ impl UserHandler {
// Change user's name // Change user's name
if let Some(user) = self.state.online_users.write().unwrap().get_mut(&addr) { if let Some(user) = self.state.online_users.write().unwrap().get_mut(&addr) {
user.write().unwrap().change_name(request.username); user.write()
.unwrap()
.change_name(request.username[..username_max_len].to_string());
} else { } else {
tracing::error!("Error updating username: Can't find user!"); tracing::error!("Error updating username: Can't find user!");
return; return;