phil/ezidam
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

avatar: return 404 if user does not exist

Browse source
This commit is contained in:
Philippe Loctaux 2023-03-08 23:11:33 +01:00
parent 1fd7901037
commit 5b8fef624d
2 changed files with 7 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

4
crates/ezidam/src/error.rs
View file

@ -39,4 +39,8 @@ impl Error {
pub fn internal_server_error<E: std::error::Error>(error: E) -> Self { pub fn internal_server_error<E: std::error::Error>(error: E) -> Self {
Self::new(Status::InternalServerError, error) Self::new(Status::InternalServerError, error)
} }
pub fn not_found<M: std::fmt::Display>(value: M) -> Self {
Self::new(Status::NotFound, value)
}
} }

4
crates/ezidam/src/routes/root.rs
View file

@ -47,7 +47,9 @@ async fn avatar(
size: Option<u32>, size: Option<u32>,
) -> Result<FileFromBytes> { ) -> Result<FileFromBytes> {
// Verify existence of user // Verify existence of user
let _user = User::get_by_id(&mut *db, &user_id.0).await?; let _user = User::get_by_id(&mut *db, &user_id.0)
.await?
.ok_or_else(|| Error::not_found(user_id.0.to_string()))?;
// Generate avatar // Generate avatar
let avatar = task::spawn_blocking(move || { let avatar = task::spawn_blocking(move || {