fix: Make sure algorithm is being respected in data_hash.rs (#613)

Make sure algorithm is being respected in data_hash.rs
contentauth · Oct 4, 2024 · b48e574 · b48e574
1 parent 53a02ea
commit b48e574
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/sdk/src/assertions/data_hash.rs b/sdk/src/assertions/data_hash.rs
@@ -219,11 +219,17 @@ impl DataHash {
             return Err(Error::BadParam("asset hash is remote".to_owned()));
         }
 
-        let curr_alg = alg.unwrap_or("sha256");
+        let curr_alg = match &self.alg {
+            Some(a) => a.clone(),
+            None => match alg {
+                Some(a) => a.to_owned(),
+                None => return Err(Error::HashMismatch("no alg specified".to_owned())),
+            },
+        };
 
         let exclusions = self.exclusions.as_ref().cloned();
 
-        if verify_asset_by_alg(curr_alg, &self.hash, asset_path, exclusions) {
+        if verify_asset_by_alg(&curr_alg, &self.hash, asset_path, exclusions) {
             Ok(())
         } else {
             Err(Error::HashMismatch("Hashes do not match".to_owned()))
@@ -240,7 +246,7 @@ impl DataHash {
             Some(a) => a.clone(),
             None => match alg {
                 Some(a) => a.to_owned(),
-                None => "sha256".to_string(),
+                None => return Err(Error::HashMismatch("no alg specified".to_owned())),
             },
         };