clippy: auto fix

Author: da2ce7

src/auth.rs

@@ -29,9 +29,7 @@ impl AuthorizationService {
 
         let claims = UserClaims { user, exp: exp_date };
 
-        let token = encode(&Header::default(), &claims, &EncodingKey::from_secret(key)).unwrap();
-
-        token
+        encode(&Header::default(), &claims, &EncodingKey::from_secret(key)).unwrap()
     }
 
     pub async fn verify_jwt(&self, token: &str) -> Result<UserClaims, ServiceError> {

src/config.rs

@@ -135,9 +135,9 @@ impl Configuration {
             eprintln!("Creating config file..");
             let config = Configuration::default();
             let _ = config.save_to_file().await;
-            return Err(ConfigError::Message(format!(
-                "Please edit the config.TOML in the root folder and restart the tracker."
-            )));
+            return Err(ConfigError::Message(
+                "Please edit the config.TOML in the root folder and restart the tracker.".to_string(),
+            ));
         }
 
         let torrust_config: TorrustConfig = match config.try_into() {

src/databases/database.rs

@@ -11,7 +11,7 @@ use crate::models::tracker_key::TrackerKey;
 use crate::models::user::{User, UserAuthentication, UserCompact, UserProfile};
 
 /// Database drivers.
-#[derive(PartialEq, Debug, Clone, Serialize, Deserialize)]
+#[derive(PartialEq, Eq, Debug, Clone, Serialize, Deserialize)]
 pub enum DatabaseDriver {
     Sqlite3,
     Mysql,

src/databases/mysql.rs

@@ -323,7 +323,7 @@ impl Database for MysqlDatabase {
                     i += 1;
                 }
             }
-            if category_filters.len() > 0 {
+            if !category_filters.is_empty() {
                 format!(
                     "INNER JOIN torrust_categories tc ON tt.category_id = tc.category_id AND ({}) ",
                     category_filters

src/databases/sqlite.rs

@@ -318,7 +318,7 @@ impl Database for SqliteDatabase {
                     i += 1;
                 }
             }
-            if category_filters.len() > 0 {
+            if !category_filters.is_empty() {
                 format!(
                     "INNER JOIN torrust_categories tc ON tt.category_id = tc.category_id AND ({}) ",
                     category_filters

src/errors.rs

@@ -10,7 +10,7 @@ use crate::databases::database::DatabaseError;
 
 pub type ServiceResult<V> = Result<V, ServiceError>;
 
-#[derive(Debug, Display, PartialEq, Error)]
+#[derive(Debug, Display, PartialEq, Eq, Error)]
 #[allow(dead_code)]
 pub enum ServiceError {
     #[display(fmt = "internal server error")]
@@ -182,7 +182,6 @@ impl ResponseError for ServiceError {
         HttpResponseBuilder::new(self.status_code())
             .append_header((header::CONTENT_TYPE, "application/json; charset=UTF-8"))
             .body(serde_json::to_string(&ErrorToResponse { error: self.to_string() }).unwrap())
-            .into()
     }
 }
 

src/models/torrent.rs

@@ -4,7 +4,7 @@ use crate::models::torrent_file::Torrent;
 use crate::routes::torrent::CreateTorrent;
 
 #[allow(dead_code)]
-#[derive(Debug, PartialEq, Serialize, Deserialize, sqlx::FromRow)]
+#[derive(Debug, PartialEq, Eq, Serialize, Deserialize, sqlx::FromRow)]
 pub struct TorrentListing {
     pub torrent_id: i64,
     pub uploader: String,

src/models/torrent_file.rs

@@ -6,18 +6,18 @@ use sha1::{Digest, Sha1};
 use crate::config::Configuration;
 use crate::utils::hex::{bytes_to_hex, hex_to_bytes};
 
-#[derive(PartialEq, Debug, Clone, Serialize, Deserialize)]
+#[derive(PartialEq, Eq, Debug, Clone, Serialize, Deserialize)]
 pub struct TorrentNode(String, i64);
 
-#[derive(PartialEq, Debug, Clone, Serialize, Deserialize)]
+#[derive(PartialEq, Eq, Debug, Clone, Serialize, Deserialize)]
 pub struct TorrentFile {
     pub path: Vec<String>,
     pub length: i64,
     #[serde(default)]
     pub md5sum: Option<String>,
 }
 
-#[derive(PartialEq, Debug, Clone, Serialize, Deserialize)]
+#[derive(PartialEq, Eq, Debug, Clone, Serialize, Deserialize)]
 pub struct TorrentInfo {
     pub name: String,
     #[serde(default)]
@@ -160,7 +160,7 @@ impl Torrent {
 
     pub fn file_size(&self) -> i64 {
         if self.info.length.is_some() {
-            return self.info.length.unwrap();
+            self.info.length.unwrap()
         } else {
             match &self.info.files {
                 None => 0,
@@ -176,15 +176,15 @@ impl Torrent {
     }
 }
 
-#[derive(PartialEq, Debug, Clone, Serialize, Deserialize, sqlx::FromRow)]
+#[derive(PartialEq, Eq, Debug, Clone, Serialize, Deserialize, sqlx::FromRow)]
 pub struct DbTorrentFile {
     pub path: Option<String>,
     pub length: i64,
     #[serde(default)]
     pub md5sum: Option<String>,
 }
 
-#[derive(PartialEq, Debug, Clone, Serialize, Deserialize, sqlx::FromRow)]
+#[derive(PartialEq, Eq, Debug, Clone, Serialize, Deserialize, sqlx::FromRow)]
 pub struct DbTorrentInfo {
     pub name: String,
     pub pieces: String,
@@ -194,7 +194,7 @@ pub struct DbTorrentInfo {
     pub root_hash: i64,
 }
 
-#[derive(PartialEq, Debug, Clone, Serialize, Deserialize, sqlx::FromRow)]
+#[derive(PartialEq, Eq, Debug, Clone, Serialize, Deserialize, sqlx::FromRow)]
 pub struct DbTorrentAnnounceUrl {
     pub tracker_url: String,
 }

src/models/user.rs

@@ -13,7 +13,7 @@ pub struct UserAuthentication {
     pub password_hash: String,
 }
 
-#[derive(Debug, PartialEq, Serialize, Deserialize, Clone, sqlx::FromRow)]
+#[derive(Debug, PartialEq, Eq, Serialize, Deserialize, Clone, sqlx::FromRow)]
 pub struct UserProfile {
     pub user_id: i64,
     pub username: String,

src/routes/category.rs

@@ -57,7 +57,7 @@ pub async fn delete_category(
         return Err(ServiceError::Unauthorized);
     }
 
-    let _ = app_data.database.delete_category(&payload.name).await?;
+    app_data.database.delete_category(&payload.name).await?;
 
     Ok(HttpResponse::Ok().json(OkResponse {
         data: payload.name.clone(),