Ling0925
/
buster
mirror of https://github.com/buster-so/buster.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Refactor permission handling in get_message_with_permission function 

- Updated the logic to determine final_permission by checking if permission is Some, simplifying the condition.
- Improved error handling for non-public threads when no permissions are provided.
This commit is contained in:
dal 2025-01-06 15:21:37 -07:00
parent c77018abbd
commit a7ae1b1dfd
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
api/src/routes/ws/threads_and_messages/messages_utils.rs
View File

@ -84,7 +84,9 @@ pub async fn get_message_with_permission(
} }
}; };
let final_permission = if permission.is_none() && !is_public_thread { let final_permission = if permission.is_some() {
permission.unwrap()
} else if !is_public_thread {
return Err(anyhow!("No message found with permissions")); return Err(anyhow!("No message found with permissions"));
} else { } else {
AssetPermissionRole::Viewer AssetPermissionRole::Viewer