From 841fc1355ecef4636ad9f5ad6d081f72eeb868ac Mon Sep 17 00:00:00 2001 From: Owen Jacobson Date: Wed, 21 May 2025 22:43:52 -0400 Subject: 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" scenario into an internal server error. --- src/user/routes/logout/post.rs | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) (limited to 'src/user') 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() + } } } } -- cgit v1.2.3