remove mutex and use u16 type for port

fl-server/src/config.rs

@@ -27,7 +27,7 @@ pub struct AppState {
 #[derive(Debug, Default, Clone, Deserialize)]
 pub struct Config {
     pub host: String,
-    pub port: usize,
+    pub port: u16,
     pub store_url: Vec<String>,
     pub flist_dir: String,
 
@@ -45,13 +45,6 @@ pub async fn parse_config(filepath: &str) -> Result<Config> {
         anyhow::bail!("host '{}' is invalid", c.host)
     }
 
-    if c.port > 65535 {
-        anyhow::bail!(format!(
-            "port '{}' is invalid, must be between [0, 65535]",
-            c.port
-        ))
-    }
-
     rfs::store::parse_router(&c.store_url)
         .await
         .context("failed to parse store urls")?;

fl-server/src/db.rs

@@ -1,4 +1,4 @@
-use std::{collections::HashMap, sync::Mutex};
+use std::collections::HashMap;
 
 use serde::{Deserialize, Serialize};
 use utoipa::ToSchema;
@@ -15,29 +15,22 @@ pub trait DB: Send + Sync {
 
 #[derive(Debug, ToSchema)]
 pub struct MapDB {
-    users: Mutex<HashMap<String, User>>,
+    users: HashMap<String, User>,
 }
 
 impl MapDB {
     pub fn new(users: &[User]) -> Self {
         Self {
-            users: Mutex::new(
-                users
-                    .to_vec()
-                    .iter()
-                    .map(|u| (u.username.clone(), u.to_owned()))
-                    .collect(),
-            ),
+            users: users
+                .iter()
+                .map(|u| (u.username.clone(), u.to_owned()))
+                .collect(),
         }
     }
 }
 
 impl DB for MapDB {
     fn get_user_by_username(&self, username: &str) -> Option<User> {
-        self.users
-            .lock()
-            .expect("failed to lock users map")
-            .get(username)
-            .cloned()
+        self.users.get(username).cloned()
     }
 }