feat: Add port and request payload limit in .env

- Added `PORT` to specify the server listening port. - Added
`OHKAMI_REQUEST_PAYLOAD_LIMIT` for security.

Author: TOwInOK

.env

@@ -1,2 +1,4 @@
 STORE_PATH="./.store"
 CACHE_PATH="./.cache"
+PORT="3000"
+OHKAMI_REQUEST_PAYLOAD_LIMIT=4046

src/app.rs

@@ -57,6 +57,7 @@ pub struct AppState {
 pub async fn app(
     lock: Lock,
     cache: Cache,
+    address: &str,
 ) {
     let router = Ohkami::new((
         Context::new(Arc::new(AppState {
@@ -81,5 +82,5 @@ pub async fn app(
     .expect("wtf");
 
     let router = Ohkami::new(("/doc".GET(doc), "/".By(router)));
-    router.howl("localhost:3000").await;
+    router.howl(address).await;
 }

src/app/skin.rs

@@ -27,8 +27,8 @@ pub fn skin_router() -> Ohkami {
 }
 
 #[derive(Debug, Clone, Deserialize, Schema, Hash, PartialEq, Eq, Serialize)]
-/// Params for expose Tee
 pub struct SkinQuery {
+    /// Replace all whitespaces to `_`
     pub name: String,
     pub body: Option<u32>,
     pub feet: Option<u32>,

src/error.rs

@@ -4,7 +4,7 @@ use ohkami::{IntoResponse, Response};
 use reqwest::header::ToStrError;
 use thiserror::Error;
 use tokio::{io, task::JoinError};
-use tracing::error;
+use tracing::{error, instrument};
 
 #[derive(Debug, Error)]
 pub enum Error {
@@ -33,39 +33,27 @@ pub enum Error {
 }
 
 impl IntoResponse for Error {
+    #[instrument]
     fn into_response(self) -> Response {
         match self {
-            Error::Io(e) => {
-                error!(error=%e,"I/O error");
-                Response::InternalServerError().with_text("Skin not found")
-            }
-            Error::Tee(e) => {
-                error!(error=%e,"Tee error");
-                Response::InternalServerError().with_text("Fail to render uv")
-            }
+            Error::Io(_) => Response::InternalServerError().with_text("Skin not found"),
+            Error::Tee(_) => Response::InternalServerError().with_text("Fail to render uv"),
             Error::QueryNameNotFound => {
-                error!("Query expected name, but got none");
                 Response::BadRequest().with_text("Query expected name, but got none")
             }
-            Error::Reqwest(e) => {
-                error!(error=%e,"Reqwest error");
-                Response::InternalServerError()
-            }
-            Error::ToStrError(e) => {
-                error!(error=%e,"Reqwest ToStr error");
-                Response::InternalServerError()
-            }
-            Error::SaveFailed {
-                path,
-                name,
-                error,
-            } => todo!(),
+            Error::Reqwest(_) => Response::InternalServerError(),
+            Error::ToStrError(_) => Response::InternalServerError(),
             Error::DownloadFailed {
-                name,
-                error,
+                name: _,
+                error: _,
             } => todo!(),
-            Error::TaskJoin(join_error) => todo!(),
-            Error::Json(error) => todo!(),
+            Error::TaskJoin(_join_error) => Response::InternalServerError(),
+            Error::Json(_error) => Response::InternalServerError(),
+            Error::SaveFailed {
+                path: _,
+                name: _,
+                error: _,
+            } => Response::InternalServerError(),
         }
     }
 }

src/main.rs

@@ -36,6 +36,10 @@ async fn main() {
     dotenvy::dotenv().ok();
 
     init_logger(Level::INFO);
+    let address = format!(
+        "0.0.0.0:{}",
+        std::env::var("PORT").expect("PORT must be set")
+    );
     let lock = Arc::new(
         LockStore::read(std::env::var("STORE_PATH").expect("STORE_PATH must be set"))
             .await
@@ -81,5 +85,5 @@ async fn main() {
         });
     }
 
-    app(lock, cache).await
+    app(lock, cache, &address).await
 }