Make Cip509 field private

Author: stanislav-tkach

rust/rbac-registration/src/cardano/cip509/mod.rs

@@ -49,19 +49,19 @@ pub struct Cip509 {
     /// A registration purpose (`UUIDv4`).
     ///
     /// The purpose is defined by the consuming dApp.
-    pub purpose: Uuid,
+    purpose: Uuid,
     /// Transaction inputs hash.
-    pub txn_inputs_hash: TxInputHash,
+    txn_inputs_hash: TxInputHash,
     /// An optional hash of the previous transaction.
     ///
     /// The hash must always be present except for the first registration transaction.
-    pub prv_tx_id: Option<Blake2b256Hash>,
+    prv_tx_id: Option<Blake2b256Hash>,
     /// Metadata.
     ///
     /// This field encoded in chunks. See [`X509Chunks`] for more details.
-    pub metadata: Cip509RbacMetadata,
+    metadata: Cip509RbacMetadata,
     /// Validation signature.
-    pub validation_signature: ValidationSignature,
+    validation_signature: ValidationSignature,
 }
 
 /// Validation value for CIP509 metadatum.
@@ -239,4 +239,19 @@ impl Cip509 {
             additional_data: AdditionalData { precomputed_aux },
         }
     }
+
+    /// Returns a registration purpose.
+    pub fn purpose(&self) -> Uuid {
+        self.purpose
+    }
+
+    /// Returns an identifier of the previous transaction.
+    pub fn previous_transaction(&self) -> Option<&Blake2b256Hash> {
+        self.prv_tx_id.as_ref()
+    }
+
+    /// Returns CIP509 metadata.
+    pub fn metadata(&self) -> &Cip509RbacMetadata {
+        &self.metadata
+    }
 }

rust/rbac-registration/src/registration/cardano/mod.rs

@@ -185,7 +185,7 @@ impl RegistrationChainInner {
         txn: &MultiEraTx,
     ) -> anyhow::Result<Self> {
         // Should be chain root, return immediately if not
-        if cip509.prv_tx_id.is_some() {
+        if cip509.previous_transaction().is_some() {
             bail!("Invalid chain root, previous transaction ID should be None.");
         }
 
@@ -200,9 +200,9 @@ impl RegistrationChainInner {
         }
 
         // Add purpose to the list
-        let purpose = vec![cip509.purpose];
+        let purpose = vec![cip509.purpose()];
 
-        let registration = cip509.metadata;
+        let registration = cip509.metadata().clone();
         let point_tx_idx = PointTxIdx::new(point, tx_idx);
 
         let certificate_uris = registration.certificate_uris;
@@ -259,23 +259,23 @@ impl RegistrationChainInner {
         }
 
         // Check and update the current transaction ID hash
-        if let Some(prv_tx_id) = cip509.prv_tx_id {
+        if let Some(prv_tx_id) = cip509.previous_transaction() {
             // Previous transaction ID in the CIP509 should equal to the current transaction ID
             // or else it is not a part of the chain
-            if prv_tx_id == self.current_tx_id_hash {
-                new_inner.current_tx_id_hash = prv_tx_id;
+            if prv_tx_id == &self.current_tx_id_hash {
+                new_inner.current_tx_id_hash = prv_tx_id.clone();
             } else {
                 bail!("Invalid previous transaction ID, not a part of this registration chain");
             }
         }
 
         // Add purpose to the chain, if not already exist
-        let purpose = cip509.purpose;
+        let purpose = cip509.purpose();
         if !self.purpose.contains(&purpose) {
             new_inner.purpose.push(purpose);
         }
 
-        let registration = cip509.metadata;
+        let registration = cip509.metadata().clone();
         let point_tx_idx = PointTxIdx::new(point, tx_idx);
         new_inner.certificate_uris = new_inner.certificate_uris.update(&registration);
         update_x509_certs(&mut new_inner, registration.x509_certs, &point_tx_idx);