fix(server): `auth/register` handle panic
This commit is contained in:
parent
5f920d66cf
commit
282731d260
|
@ -40,7 +40,7 @@ impl Database {
|
||||||
options.log_statements(LevelFilter::Debug);
|
options.log_statements(LevelFilter::Debug);
|
||||||
|
|
||||||
// create a database pool
|
// create a database pool
|
||||||
let conn = SqlitePool::connect_with(options.clone())
|
let conn = SqlitePool::connect_with(options)
|
||||||
.await
|
.await
|
||||||
.map_err(Error::ConnectDatabase)?;
|
.map_err(Error::ConnectDatabase)?;
|
||||||
|
|
||||||
|
|
|
@ -5,7 +5,7 @@ use homedisk_database::{Database, User};
|
||||||
use homedisk_types::{
|
use homedisk_types::{
|
||||||
auth::login::{Request, Response},
|
auth::login::{Request, Response},
|
||||||
config::Config,
|
config::Config,
|
||||||
errors::{AuthError, ServerError},
|
errors::{AuthError, FsError, ServerError},
|
||||||
};
|
};
|
||||||
|
|
||||||
use crate::middleware::{create_token, validate_json};
|
use crate::middleware::{create_token, validate_json};
|
||||||
|
@ -62,7 +62,8 @@ pub async fn handle(
|
||||||
storage = config.storage.path,
|
storage = config.storage.path,
|
||||||
username = user.username,
|
username = user.username,
|
||||||
);
|
);
|
||||||
fs::create_dir_all(&user_dir).unwrap();
|
fs::create_dir_all(&user_dir)
|
||||||
|
.map_err(|e| ServerError::FsError(FsError::CreateDirectory(e.to_string())))?;
|
||||||
|
|
||||||
Ok(Json(response))
|
Ok(Json(response))
|
||||||
}
|
}
|
||||||
|
|
|
@ -44,7 +44,6 @@ pub async fn handle(
|
||||||
// delete file
|
// delete file
|
||||||
if path.is_file() {
|
if path.is_file() {
|
||||||
fs::remove_file(&path)
|
fs::remove_file(&path)
|
||||||
// return error
|
|
||||||
.map_err(|err| ServerError::FsError(FsError::DeleteFile(err.to_string())))?;
|
.map_err(|err| ServerError::FsError(FsError::DeleteFile(err.to_string())))?;
|
||||||
}
|
}
|
||||||
// delete directory
|
// delete directory
|
||||||
|
|
|
@ -28,7 +28,7 @@ pub enum Error {
|
||||||
#[derive(Debug, Clone, Serialize, Deserialize)]
|
#[derive(Debug, Clone, Serialize, Deserialize)]
|
||||||
#[serde(rename_all = "kebab-case")]
|
#[serde(rename_all = "kebab-case")]
|
||||||
enum ResponseError {
|
enum ResponseError {
|
||||||
Error(String)
|
Error(String),
|
||||||
}
|
}
|
||||||
|
|
||||||
impl Error {
|
impl Error {
|
||||||
|
|
Loading…
Reference in New Issue