chore: make AI review happy

Author: bxf12315

.dockerignore

@@ -0,0 +1,7 @@
+.idea
+.DS_Store
+/data
+.trustify
+/target
+/.dockerignore
+/Containerfile

.gitignore

@@ -18,4 +18,4 @@
 /dump.sql
 .ai_history.txt
 /vendor/
-/vendor.toml
+/vendor.toml

Cargo.lock

@@ -110,7 +110,7 @@ petgraph = { version = "0.8.0", features = ["serde-1"] }
 quick-xml = "0.39.0"
 rand = "0.10.0"
 regex = "1.10.3"
-reqwest = { version = "0.13.1", features = ["blocking", "form", "json", "query"] }
+reqwest = "0.13"
 ring = "0.17.8"
 roxmltree = "0.21.1"
 rstest = "0.26.1"
@@ -213,4 +213,4 @@ csaf = { git = "https://github.com/scm-rs/csaf-rs", branch = "main" }
 #osv = { git = "https://github.com/ctron/osv", branch = "feature/drop_deps_1" }
 
 # required due to https://github.com/doubleopen-project/spdx-rs/pull/35
-spdx-rs = { git = "https://github.com/ctron/spdx-rs", branch = "feature/add_alias_2" }
+spdx-rs = { git = "https://github.com/ctron/spdx-rs", branch = "feature/add_alias_2" }

Cargo.toml

@@ -198,4 +198,4 @@
    distributed under the License is distributed on an "AS IS" BASIS,
    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    See the License for the specific language governing permissions and
-   limitations under the License.
+   limitations under the License.

LICENSE

@@ -279,4 +279,4 @@ CPE (Product?) and/or pURLs described by the SBOM
 ## Related links
 
 * [Trustify scale test results](https://guacsec.github.io/trustify-scale-test-runs/)
-* [Trustify benchmark results](https://guacsec.github.io/trustify/dev/bench/)
+* [Trustify benchmark results](https://guacsec.github.io/trustify/dev/bench/)

README.md

@@ -13,7 +13,7 @@ clap = { workspace = true, features = ["derive", "env"] }
 dotenvy = { workspace = true }
 futures = { workspace = true }
 indicatif = { workspace = true }
-reqwest = { workspace = true, features = ["blocking", "json"] }
+reqwest = { workspace = true, features = ["form"] }
 serde = { workspace = true , features = ["derive"] }
 serde_json = { workspace = true }
 thiserror = { workspace = true }

etc/trustify-cli/Cargo.toml

@@ -18,7 +18,7 @@ pub enum ApiError {
     #[error("HTTP {0}: {1}")]
     HttpError(u16, String),
 
-    #[error("HTTP 404: Resource not found")]
+    #[error("HTTP 404: Resource not found {0}")]
     NotFound(String),
 
     #[error("HTTP 401: Please check your authentication credentials")]
@@ -43,7 +43,7 @@ pub enum ApiError {
 impl From<reqwest::Error> for ApiError {
     fn from(e: reqwest::Error) -> Self {
         if e.is_timeout() {
-            ApiError::Timeout(0) // 0 indicates network-level timeout (no HTTP response)
+            ApiError::Timeout(e.status().unwrap_or(StatusCode::REQUEST_TIMEOUT).as_u16()) // 0 indicates network-level timeout (no HTTP response)
         } else if e.is_connect() {
             ApiError::NetworkError(format!("Connection failed: {}", e))
         } else if e.is_request() {

etc/trustify-cli/src/api/client.rs

@@ -291,7 +291,9 @@ fn format_list_output(json: &str, format: &ListFormat) -> anyhow::Result<ExitCod
                     })
                 })
                 .collect();
-            println!("{}", serde_json::to_string(&result).unwrap_or_default());
+            let json = serde_json::to_string(&result)
+                .map_err(|e| anyhow::anyhow!("Failed to serialize output: {}", e))?;
+            println!("{}", json);
             Ok(ExitCode::SUCCESS)
         }
         ListFormat::Short => {
@@ -308,7 +310,9 @@ fn format_list_output(json: &str, format: &ListFormat) -> anyhow::Result<ExitCod
                     })
                 })
                 .collect();
-            println!("{}", serde_json::to_string(&result).unwrap_or_default());
+            let json = serde_json::to_string(&result)
+                .map_err(|e| anyhow::anyhow!("Failed to serialize output: {}", e))?;
+            println!("{}", json);
             Ok(ExitCode::SUCCESS)
         }
     }