refactor(importer): extract trakt client id to env variable

Author: IgnisDa

.github/workflows/main.yml

@@ -174,6 +174,7 @@ jobs:
           APP_VERSION: ${{ steps.build.outputs.version }}
           DEFAULT_TMDB_ACCESS_TOKEN: ${{ secrets.DEFAULT_TMDB_ACCESS_TOKEN }}
           DEFAULT_MAL_CLIENT_ID: ${{ secrets.DEFAULT_MAL_CLIENT_ID }}
+          TRAKT_CLIENT_ID: ${{ secrets.TRAKT_CLIENT_ID }}
           UNKEY_API_ID: ${{ secrets.UNKEY_API_ID }}
         run: |
           ${{ matrix.platform.command }} build --locked --target ${{ matrix.platform.target }} --release

Cargo.lock

@@ -3,6 +3,7 @@ passthrough = [
     "APP_VERSION",
     "DEFAULT_TMDB_ACCESS_TOKEN",
     "DEFAULT_MAL_CLIENT_ID",
+    "TRAKT_CLIENT_ID",
     "UNKEY_API_ID",
 ]
 

Cross.toml

@@ -15,6 +15,7 @@ convert_case = { workspace = true }
 csv = "=1.3.0"
 data-encoding = "=2.6.0"
 enum_meta = { workspace = true }
+env-utils = { path = "../../utils/env" }
 common-models = { path = "../../models/common" }
 common-utils = { path = "../../utils/common" }
 database-models = { path = "../../models/database" }

crates/services/importer/Cargo.toml

@@ -4,6 +4,7 @@ use common_utils::{ryot_log, APPLICATION_JSON_HEADER};
 use convert_case::{Case, Casing};
 use dependent_models::ImportResult;
 use enums::{ImportSource, MediaLot, MediaSource};
+use env_utils::TRAKT_CLIENT_ID;
 use itertools::Itertools;
 use media_models::{
     CreateOrUpdateCollectionInput, DeployTraktImportInput, ImportOrExportItemRating,
@@ -18,7 +19,6 @@ use serde::{Deserialize, Serialize};
 use super::{ImportFailStep, ImportFailedItem, ImportOrExportMediaItem};
 
 const API_URL: &str = "https://api.trakt.tv";
-const CLIENT_ID: &str = "b3d93fd4c53d78d61b18e0f0bf7ad5153de323788dbc0be1a3627205a36e89f5";
 const API_VERSION: &str = "2";
 
 #[derive(Debug, Serialize, Deserialize)]
@@ -63,7 +63,7 @@ pub async fn import(input: DeployTraktImportInput) -> Result<ImportResult> {
         (CONTENT_TYPE, APPLICATION_JSON_HEADER.clone()),
         (
             HeaderName::from_static("trakt-api-key"),
-            HeaderValue::from_static(CLIENT_ID),
+            HeaderValue::from_static(TRAKT_CLIENT_ID),
         ),
         (
             HeaderName::from_static("trakt-api-version"),

crates/services/importer/src/trakt.rs

@@ -2,3 +2,4 @@ pub const APP_VERSION: &str = env!("APP_VERSION");
 pub const UNKEY_API_ID: &str = env!("UNKEY_API_ID");
 pub const DEFAULT_TMDB_ACCESS_TOKEN: &str = env!("DEFAULT_TMDB_ACCESS_TOKEN");
 pub const DEFAULT_MAL_CLIENT_ID: &str = env!("DEFAULT_MAL_CLIENT_ID");
+pub const TRAKT_CLIENT_ID: &str = env!("TRAKT_CLIENT_ID");

crates/utils/env/src/lib.rs

@@ -21,6 +21,7 @@ UNKEY_API_ID=api_4GvvJVbWobkNjcnnvFHmBP5pXb4K
 APP_VERSION=v5.2.1
 DEFAULT_TMDB_ACCESS_TOKEN=your-tmdb-access-token
 DEFAULT_MAL_CLIENT_ID=your-mal-client-id
+TRAKT_CLIENT_ID=your-trakt-client-id
 ```
 
 In development, both servers are started independently running on `:3000` and `:5000`