From c95b45a346d34b49184b31d4080b552904d59231 Mon Sep 17 00:00:00 2001 From: Mateusz Jasiuk Date: Fri, 2 Aug 2024 09:53:27 +0200 Subject: [PATCH] feat: use inner tx hash instead of commitment hash (#92) --- shared/src/transaction.rs | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/shared/src/transaction.rs b/shared/src/transaction.rs index 4485a427..fc646290 100644 --- a/shared/src/transaction.rs +++ b/shared/src/transaction.rs @@ -11,7 +11,8 @@ use namada_tx::data::pos::{ Bond, ClaimRewards, CommissionChange, MetaDataChange, Redelegation, Unbond, Withdraw, }; -use namada_tx::data::TxType; +use namada_tx::data::{compute_inner_tx_hash, TxType}; +use namada_tx::either::Either; use namada_tx::{Section, Tx}; use serde::Serialize; @@ -274,7 +275,10 @@ impl Transaction { for (index, tx_commitment) in transaction.header().batch.into_iter().enumerate() { - let inner_tx_id = Id::from(tx_commitment.get_hash()); + let inner_tx_id = Id::from(compute_inner_tx_hash( + Some(&transaction.header_hash()), + Either::Right(&tx_commitment), + )); let memo = transaction.memo(&tx_commitment).map(|memo_bytes| {