Lint files

Author: lucemans

app/src/cache.rs

@@ -24,3 +24,9 @@ impl AppCache {
         }
     }
 }
+
+impl Default for AppCache {
+    fn default() -> Self {
+        Self::new()
+    }
+}

app/src/models/bm/player.rs

@@ -1,9 +1,6 @@
-use std::ops::Deref;
-
-use chrono::DateTime;
 use poem::Result;
-use poem_openapi::{payload::Json, types::ToJSON, Object, OpenApi};
-use reqwest::{Client, ClientBuilder};
+use poem_openapi::Object;
+use reqwest::ClientBuilder;
 use serde::{Deserialize, Serialize};
 use tracing::{info, warn};
 

app/src/models/bm/recent.rs

@@ -1,7 +1,7 @@
 use chrono::DateTime;
 use poem::Result;
-use poem_openapi::{Object, OpenApi};
-use reqwest::{Client, ClientBuilder};
+use poem_openapi::Object;
+use reqwest::ClientBuilder;
 use serde::{Deserialize, Serialize};
 use tracing::{info, warn};
 
@@ -56,7 +56,7 @@ impl BattleMetricsRecentServer {
             .relationships
             .as_ref()
             .and_then(|r| r.game.as_ref())
-            .map_or(false, |d| d.data._type == "game" && d.data.id == "rust");
+            .is_some_and(|d| d.data._type == "game" && d.data.id == "rust");
 
         if !is_rust {
             return None;
@@ -84,15 +84,11 @@ impl BattleMetricsRecentServer {
             tags: value.attributes.as_ref().and_then(|a| {
                 a.extra.get("details").and_then(|d| {
                     d.get("tags").and_then(|v| {
-                        if let Some(arr) = v.as_array() {
-                            Some(
-                                arr.iter()
-                                    .filter_map(|item| item.as_str().map(|s| s.to_string()))
-                                    .collect(),
-                            )
-                        } else {
-                            None
-                        }
+                        v.as_array().map(|arr| 
+                            arr.iter()
+                                .filter_map(|item| item.as_str().map(|s| s.to_string()))
+                                .collect()
+                        )
                     })
                 })
             }),

app/src/server/auth/mod.rs

@@ -52,8 +52,8 @@ impl AuthApi {
             })?;
 
         // Check if it starts with "Bearer " and extract the token
-        let token = if auth_header.starts_with("Bearer ") {
-            auth_header[7..].to_string()
+        let token = if let Some(token_str) = auth_header.strip_prefix("Bearer ") {
+            token_str.to_string()
         } else {
             return Err(poem::Error::from_string(
                 "Invalid Authorization header format. Expected 'Bearer TOKEN'",

app/src/server/auth/mw.rs

@@ -1,9 +1,8 @@
-use poem::{web::Data, Error, FromRequest, Request, RequestBody, Result};
+use poem::{web::Data, FromRequest, Request, RequestBody, Result};
 use poem_openapi::{
     registry::{MetaSecurityScheme, Registry},
     ApiExtractor, ApiExtractorType, ExtractParamOptions,
 };
-use reqwest::StatusCode;
 use serde::{Deserialize, Serialize};
 
 use crate::{models::user::User, state::AppState};
@@ -94,7 +93,6 @@ impl<'a> ApiExtractor<'a> for AuthUser {
         vec!["AuthToken"]
     }
 }
-
 impl AuthUser {
     fn state(&self) -> &AppState {
         match self {
@@ -103,3 +101,4 @@ impl AuthUser {
         }
     }
 }
+

app/src/server/auth/oauth.rs

@@ -2,7 +2,7 @@ use crate::models::user::User;
 use crate::server::ApiTags;
 use crate::state::{AppState, SteamOAuthConfig};
 use async_std::task;
-use poem::{web::Query, EndpointExt, Result};
+use poem::{web::Query, Result};
 use poem_openapi::payload::Response;
 use poem_openapi::{payload::Html, payload::PlainText, OpenApi};
 use reqwest::{Client, StatusCode};

app/src/server/bm/mod.rs

@@ -1,20 +1,16 @@
 use crate::{
     models::bm::{
-        player::{get_quick_match_players, get_quick_match_players_cached},
-        recent::{get_recent_server_by_player_id, get_recent_servers_cached, BattleMetricsRecentServers},
+        player::get_quick_match_players_cached,
+        recent::{get_recent_servers_cached, BattleMetricsRecentServers},
     },
     state::AppState,
 };
-use chrono::DateTime;
 use poem::{
-    http::Error,
-    web::{Data, Query},
-    EndpointExt, Result,
+    web::Data,
+    Result,
 };
-use poem_openapi::{payload::Json, Object, OpenApi};
-use reqwest::{Client, ClientBuilder};
-use serde::{Deserialize, Serialize};
-use tracing::{info, warn};
+use poem_openapi::{payload::Json, OpenApi};
+use tracing::info;
 
 use super::auth::mw::AuthUser;
 

app/src/server/maps/mod.rs

@@ -96,7 +96,7 @@ impl MapsApi {
     #[oai(path = "/maps/search", method = "get", tag = "ApiTags::Maps")]
     async fn search(
         &self,
-        state: Data<&AppState>,
+        _state: Data<&AppState>,
         search: Query<String>,
     ) -> Result<Json<SearchResponse>> {
         let url = format!(
@@ -125,7 +125,7 @@ impl MapsApi {
     #[oai(path = "/maps/get", method = "get", tag = "ApiTags::Maps")]
     async fn get(
         &self,
-        state: Data<&AppState>,
+        _state: Data<&AppState>,
         map_id: Query<String>,
     ) -> Result<Json<MapResponse>> {
         let url = format!("https://api.rustmaps.com/internal/v1/maps/{}", map_id.0);

app/src/server/party/mod.rs

@@ -78,7 +78,7 @@ impl PartyApi {
     #[oai(path = "/party/create", method = "post", tag = "ApiTags::Party")]
     async fn create(
         &self,
-        state: Data<&AppState>,
+        _state: Data<&AppState>,
         body: Json<PartyCreateRequest>,
     ) -> Result<Json<PartyCreateResponse>> {
         tracing::info!("{:?}", body);
@@ -105,9 +105,9 @@ impl PartyApi {
     #[oai(path = "/party/:party_id/get", method = "get", tag = "ApiTags::Party")]
     async fn get(
         &self,
-        state: Data<&AppState>,
+        _state: Data<&AppState>,
         party_id: Path<String>,
-        cursor: Query<String>,
+        _cursor: Query<String>,
     ) -> Result<Json<PartyGetResponse>> {
         tracing::info!("{:?}", party_id.0);