asp/keep
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: replace matches with pattern matching for CoreError handling

Browse Source 
Co-authored-by: aider (openai/andrew/openrouter/qwen/qwen3-coder) <aider@aider.chat>
This commit is contained in:
Andrew Phillips
2025-08-25 18:21:56 -03:00
parent 6c00c2ab56
commit 8cc0cfc606
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/modes/server/api/item.rs
View File

@@ -197,8 +197,9 @@ pub async fn handle_get_item_latest_content(
Err(e) => { Err(e) => {
// Check if the error is ItemNotFoundGeneric to return 404 // Check if the error is ItemNotFoundGeneric to return 404
if let Some(core_err) = e.downcast_ref::<CoreError>() { if let Some(core_err) = e.downcast_ref::<CoreError>() {
if matches!(core_err, CoreError::ItemNotFoundGeneric) { match core_err {
return Err(StatusCode::NOT_FOUND); CoreError::ItemNotFoundGeneric => return Err(StatusCode::NOT_FOUND),
_ => {}
} }
} }
warn!("Failed to find latest item for content: {}", e); warn!("Failed to find latest item for content: {}", e);
@@ -258,8 +259,9 @@ pub async fn handle_get_item_content(
Err(e) => { Err(e) => {
// Check if the error is ItemNotFound to return 404 // Check if the error is ItemNotFound to return 404
if let Some(core_err) = e.downcast_ref::<CoreError>() { if let Some(core_err) = e.downcast_ref::<CoreError>() {
if matches!(core_err, CoreError::ItemNotFound(_)) { match core_err {
return Err(StatusCode::NOT_FOUND); CoreError::ItemNotFound(_) => return Err(StatusCode::NOT_FOUND),
_ => {}
} }
} }
warn!("Failed to get raw content for item {}: {}", item_id, e); warn!("Failed to get raw content for item {}: {}", item_id, e);