summaryrefslogtreecommitdiff
path: root/src/message
diff options
context:
space:
mode:
authorojacobson <ojacobson@noreply.codeberg.org>2025-05-27 04:09:01 +0200
committerojacobson <ojacobson@noreply.codeberg.org>2025-05-27 04:09:01 +0200
commit79d3598282fd573931e7cb14e4d3e54aad7c943c (patch)
treebeb3cd0de5077288f8a6c81373b6d0b7b737609f /src/message
parent1786c1eb9d13e570b028bef1206041f48d881cf0 (diff)
parent841fc1355ecef4636ad9f5ad6d081f72eeb868ac (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.rs3
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(),
}
}
}