Deterministic serialization JSON AD #794

Switch to JCS #794
atomicdata-dev · Jan 9, 2024 · d550e66 · d550e66
1 parent 9788930
commit d550e66
Show file tree

Hide file tree

Showing 4 changed files with 50 additions and 19 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -15,6 +15,7 @@ See [STATUS.md](server/STATUS.md) to learn more about which features will remain
 - Add `Agent::from_secret` #785
 - Make `set_propval` and `set_propval_shortname` chainable #785
 - Don't use default agent when fetching with Db #787
+- Deterministic serialization JSON AD #794
 
 ## [v0.36.1] - 2023-12-06
 

diff --git a/Cargo.lock b/Cargo.lock
diff --git a/lib/Cargo.toml b/lib/Cargo.toml
@@ -28,6 +28,7 @@ ring = "0.17.6"
 rio_api = {version = "0.8", optional = true}
 rio_turtle = {version = "0.8", optional = true}
 serde = {version = "1", features = ["derive"]}
+serde_jcs = "0.1.0"
 serde_json = "1"
 sled = {version = "0.34", optional = true, features = ["no_logs"]}
 toml = {version = "0.7", optional = true}

diff --git a/lib/src/commit.rs b/lib/src/commit.rs
@@ -511,7 +511,9 @@ impl Commit {
         commit_resource.remove_propval(urls::SIGNATURE);
         let json_obj =
             crate::serialize::propvals_to_json_ad_map(commit_resource.get_propvals(), None)?;
-        serde_json::to_string(&json_obj).map_err(|_| "Could not serialize to JSON-AD".into())
+        let json = serde_jcs::to_string(&json_obj)
+            .map_err(|e| format!("Failed to serialize Commit: {}", e))?;
+        Ok(json)
     }
 }