refactor: Simplify configuration loading and update server port handling

- Replace manual file reading in the load function with std::fs::read_to_string for improved clarity.
- Update the main function to dynamically set the server port from the environment variable, defaulting to 8000 if not specified.

Author: incubator4

src/config.rs

@@ -2,8 +2,6 @@ use clap::Parser;
 use serde::{Deserialize, Serialize};
 use std::collections::HashMap;
 use std::fmt;
-use std::fs::File;
-use std::io::Read;
 
 #[derive(Deserialize, Debug, Clone)]
 pub struct Config {
@@ -73,11 +71,7 @@ impl Args {
 }
 
 fn load(file_path: &str) -> Result<Config, Box<dyn std::error::Error>> {
-    let mut f = File::open(file_path)?;
-
-    let mut contents = String::new();
-    f.read_to_string(&mut contents)?;
-
+    let contents = std::fs::read_to_string(file_path)?;
     let config: Config = serde_yaml::from_str(&contents)?;
 
     Ok(config)

src/main.rs

@@ -21,7 +21,6 @@ use tracing_subscriber::{layer::SubscriberExt, util::SubscriberInitExt};
 #[tokio::main]
 async fn main() {
     // initialize tracing
-    config::Args::parse();
     let args = config::Args::parse();
     let config = args.load_config().expect("load config error");
 
@@ -74,8 +73,15 @@ async fn main() {
             ),
     );
 
-    // run our app with hyper, listening globally on port 3000
-    let listener = tokio::net::TcpListener::bind("0.0.0.0:8000").await.unwrap();
+    let port = std::env::var("PORT")
+        .ok()
+        .and_then(|p| p.parse().ok())
+        .unwrap_or(8000);
+
+    // run our app with hyper, listening globally on port 8000
+    let listener = tokio::net::TcpListener::bind(SocketAddr::from(([0, 0, 0, 0], port)))
+        .await
+        .unwrap();
     axum::serve(
         listener,
         app.into_make_service_with_connect_info::<SocketAddr>(),