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/user | |
| 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/user')
| -rw-r--r-- | src/user/routes/logout/post.rs | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/src/user/routes/logout/post.rs b/src/user/routes/logout/post.rs index bb09b9f..0ac663e 100644 --- a/src/user/routes/logout/post.rs +++ b/src/user/routes/logout/post.rs @@ -38,10 +38,13 @@ pub struct Error(#[from] pub app::ValidateError); impl IntoResponse for Error { fn into_response(self) -> Response { let Self(error) = self; - #[allow(clippy::match_wildcard_for_single_variants)] match error { - app::ValidateError::InvalidToken => Unauthorized.into_response(), - other => Internal::from(other).into_response(), + app::ValidateError::InvalidToken | app::ValidateError::LoginDeleted => { + Unauthorized.into_response() + } + app::ValidateError::Name(_) | app::ValidateError::Database(_) => { + Internal::from(error).into_response() + } } } } |
