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/ui | |
| 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/ui')
| -rw-r--r-- | src/ui/assets.rs | 3 | ||||
| -rw-r--r-- | src/ui/routes/login.rs | 1 |
2 files changed, 1 insertions, 3 deletions
diff --git a/src/ui/assets.rs b/src/ui/assets.rs index 142cdf9..642679b 100644 --- a/src/ui/assets.rs +++ b/src/ui/assets.rs @@ -54,10 +54,9 @@ pub enum Error { impl IntoResponse for Error { fn into_response(self) -> Response { - #[allow(clippy::match_wildcard_for_single_variants)] match self { Self::NotFound(_) => NotFound(self.to_string()).into_response(), - other => Internal::from(other).into_response(), + Self::Mime(_) => Internal::from(self).into_response(), } } } diff --git a/src/ui/routes/login.rs b/src/ui/routes/login.rs index 81a874c..39d45b9 100644 --- a/src/ui/routes/login.rs +++ b/src/ui/routes/login.rs @@ -4,7 +4,6 @@ pub mod get { ui::assets::{Asset, Assets}, }; - #[allow(clippy::unused_async)] pub async fn handler() -> Result<Asset, Internal> { Assets::index() } |
