feat(client-cli): add GitHub token support for authenticated API calls in snapshot converter command

Author: dlachaume

mithril-client-cli/src/commands/tools/snapshot_converter.rs

@@ -91,6 +91,10 @@ pub struct SnapshotConverterCommand {
     /// If set, the converted snapshot replaces the current ledger state in the `db_directory`.
     #[clap(long)]
     commit: bool,
+
+    /// GitHub token for authenticated API calls.
+    #[clap(long, env = "GITHUB_TOKEN")]
+    github_token: Option<String>,
 }
 
 impl SnapshotConverterCommand {
@@ -113,7 +117,7 @@ impl SnapshotConverterCommand {
                 )
             })?;
             let archive_path = Self::download_cardano_node_distribution(
-                ReqwestGitHubApiClient::new()?,
+                ReqwestGitHubApiClient::new(self.github_token.clone())?,
                 ReqwestHttpDownloader::new()?,
                 &self.cardano_node_version,
                 &distribution_dir,

mithril-client-cli/src/utils/github_release_retriever/reqwest.rs

@@ -9,25 +9,31 @@ use super::{GitHubRelease, GitHubReleaseRetriever};
 
 pub struct ReqwestGitHubApiClient {
     client: Client,
+    github_token: Option<String>,
 }
 
 impl ReqwestGitHubApiClient {
-    pub fn new() -> MithrilResult<Self> {
+    pub fn new(github_token: Option<String>) -> MithrilResult<Self> {
         let client = Client::builder()
             .user_agent("mithril-client")
             .build()
             .context("Failed to build Reqwest GitHub API client")?;
 
-        Ok(Self { client })
+        Ok(Self {
+            client,
+            github_token,
+        })
     }
 
     async fn download<U: IntoUrl, T: DeserializeOwned>(&self, source_url: U) -> MithrilResult<T> {
         let url = source_url
             .into_url()
             .with_context(|| "Given `source_url` is not a valid Url")?;
-        let response = self
-            .client
-            .get(url.clone())
+        let mut request = self.client.get(url.clone());
+        if let Some(token) = &self.github_token {
+            request = request.bearer_auth(token);
+        }
+        let response = request
             .send()
             .await
             .with_context(|| format!("Failed to send request to GitHub API: {}", url))?;
@@ -122,7 +128,7 @@ mod tests {
             when.method(GET).path("/endpoint");
             then.status(200).body(r#"{ "key": "value" }"#);
         });
-        let client = ReqwestGitHubApiClient::new().unwrap();
+        let client = ReqwestGitHubApiClient::new(None).unwrap();
 
         let result: FakeApiResponse = client
             .download(format!("{}/endpoint", server.base_url()))
@@ -144,7 +150,7 @@ mod tests {
             when.method(GET).path("/endpoint");
             then.status(200).body("this is not json");
         });
-        let client = ReqwestGitHubApiClient::new().unwrap();
+        let client = ReqwestGitHubApiClient::new(None).unwrap();
 
         let result: MithrilResult<FakeApiResponse> = client
             .download(format!("{}/endpoint", server.base_url()))
@@ -158,7 +164,7 @@ mod tests {
 
     #[tokio::test]
     async fn download_fails_on_invalid_url() {
-        let client = ReqwestGitHubApiClient::new().unwrap();
+        let client = ReqwestGitHubApiClient::new(None).unwrap();
 
         let result: MithrilResult<FakeApiResponse> = client.download("not a valid url").await;
 
@@ -172,7 +178,7 @@ mod tests {
             when.method(GET).path("/endpoint");
             then.status(StatusCode::INTERNAL_SERVER_ERROR.into());
         });
-        let client = ReqwestGitHubApiClient::new().unwrap();
+        let client = ReqwestGitHubApiClient::new(None).unwrap();
 
         let result: MithrilResult<FakeApiResponse> = client
             .download(format!("{}/endpoint", server.base_url()))