cjw/Oxicloud
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix(webdav): preserve correct status codes for file rename/move failures

Browse Source 
When file or folder rename/move operations failed in WebDAV handlers,
all errors were incorrectly converted to HTTP 500 Internal Server Error
using AppError::internal_error(). This masked specific error types:

- AlreadyExists errors should return 409 CONFLICT
- NotFound errors should return 404 NOT FOUND
- AccessDenied errors should return 403 FORBIDDEN

Changed error handling to use AppError::from() which preserves the
original DomainError type and maps to appropriate HTTP status codes.

Fixes handling of incorrect status code for file rename failure.
This commit is contained in:
BillionClaw
2026-03-17 16:29:21 +08:00
parent d1699357d8
commit 3c48aa1e6f
1 changed files with 7 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/interfaces/api/handlers/webdav_handler.rs
+7 -19
View File
@@ -1236,9 +1236,7 @@ async fn handle_move(
folder_service folder_service
.move_folder(&folder.id, move_dto, user.id) .move_folder(&folder.id, move_dto, user.id)
.await .await
.map_err(|e| { .map_err(AppError::from)?;
AppError::internal_error(format!("Failed to move folder: {}", e))
})?;
if folder.name != dest_folder_name { if folder.name != dest_folder_name {
let rename_dto = crate::application::dtos::folder_dto::RenameFolderDto { let rename_dto = crate::application::dtos::folder_dto::RenameFolderDto {
@@ -1247,9 +1245,7 @@ async fn handle_move(
folder_service folder_service
.rename_folder(&folder.id, rename_dto, user.id) .rename_folder(&folder.id, rename_dto, user.id)
.await .await
.map_err(|e| { .map_err(AppError::from)?;
AppError::internal_error(format!("Failed to rename folder: {}", e))
})?;
} }
} }
Ok(ResolvedResource::File(file)) => { Ok(ResolvedResource::File(file)) => {
@@ -1283,17 +1279,13 @@ async fn handle_move(
file_management_service file_management_service
.move_file(&file.id, Some(dest_parent_path.to_string())) .move_file(&file.id, Some(dest_parent_path.to_string()))
.await .await
.map_err(|e| { .map_err(AppError::from)?;
AppError::internal_error(format!("Failed to move file: {}", e))
})?;
} }
if file.name != dest_filename { if file.name != dest_filename {
file_management_service file_management_service
.rename_file(&file.id, dest_filename) .rename_file(&file.id, dest_filename)
.await .await
.map_err(|e| { .map_err(AppError::from)?;
AppError::internal_error(format!("Failed to rename file: {}", e))
})?;
} }
} }
Err(_) => { Err(_) => {
@@ -1354,9 +1346,7 @@ async fn handle_move(
folder_service folder_service
.rename_folder(&folder.id, rename_dto, user.id) .rename_folder(&folder.id, rename_dto, user.id)
.await .await
.map_err(|e| { .map_err(AppError::from)?;
AppError::internal_error(format!("Failed to rename folder: {}", e))
})?;
} }
} else { } else {
let file = file_retrieval_service let file = file_retrieval_service
@@ -1396,15 +1386,13 @@ async fn handle_move(
file_management_service file_management_service
.move_file(&file.id, Some(dest_parent_path.to_string())) .move_file(&file.id, Some(dest_parent_path.to_string()))
.await .await
.map_err(|e| AppError::internal_error(format!("Failed to move file: {}", e)))?; .map_err(AppError::from)?;
} }
if file.name != dest_filename { if file.name != dest_filename {
file_management_service file_management_service
.rename_file(&file.id, dest_filename) .rename_file(&file.id, dest_filename)
.await .await
.map_err(|e| { .map_err(AppError::from)?;
AppError::internal_error(format!("Failed to rename file: {}", e))
})?;
} }
} }
} }