From 6425d4196874014e975554f5418370108eb21923 Mon Sep 17 00:00:00 2001
From: YuevUwU <96368079+YuevUwU@users.noreply.github.com>
Date: Sat, 28 Mar 2026 00:08:18 +0800
Subject: [PATCH 1/2] fix: silent fallback to default on any server_config.yml
 error

- Use default if server_config.yml is not found
  with warn log and yaml form of default (as example for users)
- Stop if raised other io::Error or serde_yaml::Error
- TcpListener -> Server is now TryFrom
---
 phira-mp-server/src/main.rs   |  2 +-
 phira-mp-server/src/server.rs | 36 +++++++++++++++++++++++------------
 2 files changed, 25 insertions(+), 13 deletions(-)

diff --git a/phira-mp-server/src/main.rs b/phira-mp-server/src/main.rs
index d50916f..ce551c1 100644
--- a/phira-mp-server/src/main.rs
+++ b/phira-mp-server/src/main.rs
@@ -107,7 +107,7 @@ async fn main() -> Result<()> {
         println!("Local Address: {}", addr);
     }
 
-    let listener: Server = TcpListener::bind(addrs).await?.into();
+    let listener: Server = TcpListener::bind(addrs).await?.try_into()?;
 
     loop {
         if let Err(err) = listener.accept().await {
diff --git a/phira-mp-server/src/server.rs b/phira-mp-server/src/server.rs
index c67a80d..1558d60 100644
--- a/phira-mp-server/src/server.rs
+++ b/phira-mp-server/src/server.rs
@@ -1,19 +1,21 @@
 use crate::{IdMap, Room, SafeMap, Session, User, vacant_entry};
-use anyhow::Result;
+use anyhow::{Context, Result};
 use phira_mp_common::RoomId;
-use serde::Deserialize;
-use std::{fs::File, sync::Arc};
+use serde::{Deserialize, Serialize};
+use std::{fs::File, io, sync::Arc};
 use tokio::{net::TcpListener, sync::mpsc, task::JoinHandle};
 use tracing::{info, warn};
 use uuid::Uuid;
 
+const CONFIG_PATH: &str = "server_config.yml";
+
 #[derive(Debug, Deserialize)]
 pub struct Chart {
     pub id: i32,
     pub name: String,
 }
 
-#[derive(Debug, Deserialize)]
+#[derive(Debug, Deserialize, Serialize)]
 pub struct ServerConfig {
     pub monitors: Vec<i32>,
 }
@@ -55,13 +57,23 @@ pub struct Server {
     lost_con_handle: JoinHandle<()>,
 }
 
-impl From<TcpListener> for Server {
-    fn from(listener: TcpListener) -> Self {
+impl TryFrom<TcpListener> for Server {
+    type Error = anyhow::Error;
+    fn try_from(listener: TcpListener) -> Result<Server, Self::Error> {
         let (lost_con_tx, mut lost_con_rx) = mpsc::channel(16);
-        let config: ServerConfig = File::open("server_config.yml")
-            .ok()
-            .and_then(|f| serde_yaml::from_reader(f).ok())
-            .unwrap_or_default();
+        let config: ServerConfig = match File::open(CONFIG_PATH) {
+            Ok(file) => serde_yaml::from_reader(file)
+                .with_context(|| format!("Failed to parse {CONFIG_PATH}")),
+            Err(e) if e.kind() == io::ErrorKind::NotFound => {
+                let default_cfg = ServerConfig::default();
+                warn!(
+                    "{CONFIG_PATH} is not found.\nFallback to default config:\n```yaml\n{}```",
+                    serde_yaml::to_string(&default_cfg).unwrap()
+                );
+                Ok(default_cfg)
+            }
+            Err(e) => Err(e).with_context(|| format!("Failed to open {CONFIG_PATH}")),
+        }?;
         let state = Arc::new(ServerState {
             config,
             sessions: IdMap::default(),
@@ -91,12 +103,12 @@ impl From<TcpListener> for Server {
             }
         });
 
-        Self {
+        Ok(Self {
             listener,
             state,
 
             lost_con_handle,
-        }
+        })
     }
 }
 

From d180b14501377407ce5aa0ed586afccb40517c72 Mon Sep 17 00:00:00 2001
From: YuevUwU <96368079+YuevUwU@users.noreply.github.com>
Date: Sat, 28 Mar 2026 11:40:24 +0800
Subject: [PATCH 2/2] chore: log config.monitors info

---
 phira-mp-server/src/server.rs | 1 +
 1 file changed, 1 insertion(+)

diff --git a/phira-mp-server/src/server.rs b/phira-mp-server/src/server.rs
index 1558d60..5c241dc 100644
--- a/phira-mp-server/src/server.rs
+++ b/phira-mp-server/src/server.rs
@@ -74,6 +74,7 @@ impl TryFrom<TcpListener> for Server {
             }
             Err(e) => Err(e).with_context(|| format!("Failed to open {CONFIG_PATH}")),
         }?;
+        info!("User IDs allowed to monitor matches: {:?}", config.monitors);
         let state = Arc::new(ServerState {
             config,
             sessions: IdMap::default(),