diff options
| author | ojacobson <ojacobson@noreply.codeberg.org> | 2025-05-27 04:09:01 +0200 |
|---|---|---|
| committer | ojacobson <ojacobson@noreply.codeberg.org> | 2025-05-27 04:09:01 +0200 |
| commit | 79d3598282fd573931e7cb14e4d3e54aad7c943c (patch) | |
| tree | beb3cd0de5077288f8a6c81373b6d0b7b737609f /src/message | |
| parent | 1786c1eb9d13e570b028bef1206041f48d881cf0 (diff) | |
| parent | 841fc1355ecef4636ad9f5ad6d081f72eeb868ac (diff) | |
Remove a bunch of clippy suppressions.
Notably, one of them was hiding a real (if unreachable) bug, by converting a "the token you have presented is not valid because the user was deleted" scenario into an internal server error, when it should have been an authorization error.
Diffstat (limited to 'src/message')
| -rw-r--r-- | src/message/routes/message/mod.rs | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/message/routes/message/mod.rs b/src/message/routes/message/mod.rs index 4abd445..a05d344 100644 --- a/src/message/routes/message/mod.rs +++ b/src/message/routes/message/mod.rs @@ -47,7 +47,6 @@ pub mod delete { impl IntoResponse for Error { fn into_response(self) -> response::Response { let Self(error) = self; - #[allow(clippy::match_wildcard_for_single_variants)] match error { DeleteError::NotSender(_) => { (StatusCode::FORBIDDEN, error.to_string()).into_response() @@ -55,7 +54,7 @@ pub mod delete { DeleteError::NotFound(_) | DeleteError::Deleted(_) => { NotFound(error).into_response() } - other => Internal::from(other).into_response(), + DeleteError::Database(_) => Internal::from(error).into_response(), } } } |
