phoenix/icarus
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 33 Wiki Activity

Added error checking for file creation in create metadata endpoint (#154)

Browse Source 
* Added error checking for file creation in create metadata endpoint

* Code formatting

* Fixed create metadata endoint

* Added missing message in the response

* Cleanup

* Warning fixes

* Code formatting
This commit was merged in pull request #154.
This commit is contained in:
KD
2025-07-14 19:42:07 -04:00
committed by GitHub
parent fd9df41f51
commit 5a2639cb59
1 changed files with 46 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/callers/song.rs
+46 -18
View File
@@ -782,28 +782,56 @@ pub mod endpoint {
Ok(data) => { Ok(data) => {
song.data = data; song.data = data;
let dir = std::path::Path::new(&song.directory); let dir = std::path::Path::new(&song.directory);
let save_path = dir.join(&song.filename); if !dir.exists() {
println!("Creating directory");
let mut file = std::fs::File::create(&save_path).unwrap(); match std::fs::create_dir_all(dir) {
file.write_all(&song.data).unwrap(); Ok(_) => {
println!("Successfully created directory");
match song.song_path() {
Ok(_) => match super::song_db::insert(&pool, &song).await {
Ok((date_created, id)) => {
song.id = id;
song.date_created = date_created;
response.data.push(song);
(axum::http::StatusCode::OK, axum::Json(response))
} }
Err(err) => { Err(err) => {
response.message = format!("{:?} song {:?}", err.to_string(), song); eprintln!("Error: Unable to create the directory {err:?}");
(axum::http::StatusCode::BAD_REQUEST, axum::Json(response))
} }
}, }
}
let save_path = dir.join(&song.filename);
match std::fs::File::create(&save_path) {
Ok(mut file) => {
file.write_all(&song.data).unwrap();
match song.song_path() {
Ok(_) => match super::song_db::insert(&pool, &song).await {
Ok((date_created, id)) => {
song.id = id;
song.date_created = date_created;
response.message = String::from("Successful");
response.data.push(song);
(axum::http::StatusCode::OK, axum::Json(response))
}
Err(err) => {
response.message =
format!("{:?} song {:?}", err.to_string(), song);
(axum::http::StatusCode::BAD_REQUEST, axum::Json(response))
}
},
Err(err) => {
response.message = err.to_string();
(axum::http::StatusCode::BAD_REQUEST, axum::Json(response))
}
}
}
Err(err) => { Err(err) => {
response.message = err.to_string(); let song_path = song.song_path();
(axum::http::StatusCode::BAD_REQUEST, axum::Json(response)) response.message = format!(
"{err:?} Song directory: {} Filename: {} Save Path: {:?} Song Path: {:?}",
song.directory, song.filename, save_path, song_path
);
(
axum::http::StatusCode::INTERNAL_SERVER_ERROR,
axum::Json(response),
)
} }
} }
} }