about summary refs log tree commit diff
diff options
context:
space:
mode:
-rw-r--r--tvix/store/src/pathinfoservice/signing_wrapper.rs26
1 files changed, 17 insertions, 9 deletions
diff --git a/tvix/store/src/pathinfoservice/signing_wrapper.rs b/tvix/store/src/pathinfoservice/signing_wrapper.rs
index 5898a5baa463..525e60e416a8 100644
--- a/tvix/store/src/pathinfoservice/signing_wrapper.rs
+++ b/tvix/store/src/pathinfoservice/signing_wrapper.rs
@@ -50,15 +50,23 @@ where
         self.inner.get(digest).await
     }
 
-    async fn put(&self, path_info: PathInfo) -> Result<PathInfo, Error> {
-        let mut path_info = path_info.clone();
-        let mut nar_info = path_info.to_narinfo();
-        nar_info.add_signature(self.signing_key.as_ref());
-        path_info.signatures = nar_info
-            .signatures
-            .into_iter()
-            .map(|s| Signature::<String>::new(s.name().to_string(), s.bytes().to_owned()))
-            .collect();
+    async fn put(&self, mut path_info: PathInfo) -> Result<PathInfo, Error> {
+        path_info.signatures.push({
+            let mut nar_info = path_info.to_narinfo();
+            nar_info.signatures.clear();
+            nar_info.add_signature(self.signing_key.as_ref());
+
+            let s = nar_info
+                .signatures
+                .pop()
+                .expect("Tvix bug: no signature after signing op");
+            debug_assert!(
+                nar_info.signatures.is_empty(),
+                "Tvix bug: more than one signature appeared"
+            );
+
+            Signature::new(s.name().to_string(), *s.bytes())
+        });
         self.inner.put(path_info).await
     }