Skip to content

Commit 141ed84

Browse files
committed
Maybe fix toLedgerTx in presence of aux data changes
We suspect that toLedgerTx is incomplete if the cardano-api Tx was modified in it's auxiliary data.
1 parent 0aae94b commit 141ed84

File tree

1 file changed

+5
-2
lines changed
  • hydra-cardano-api/src/Hydra/Cardano/Api

1 file changed

+5
-2
lines changed

hydra-cardano-api/src/Hydra/Cardano/Api/Tx.hs

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ import Cardano.Ledger.Api (
3232
datsTxWitsL,
3333
feeTxBodyL,
3434
hashScriptTxWitsL,
35+
hashTxAuxData,
3536
inputsTxBodyL,
3637
isValidTxL,
3738
mintTxBodyL,
@@ -56,7 +57,7 @@ import Cardano.Ledger.Api qualified as Ledger
5657
import Cardano.Ledger.Babbage qualified as Ledger
5758
import Cardano.Ledger.Babbage.Tx qualified as Ledger
5859
import Cardano.Ledger.Babbage.TxWits (upgradeTxDats)
59-
import Cardano.Ledger.BaseTypes (maybeToStrictMaybe, strictMaybeToMaybe)
60+
import Cardano.Ledger.BaseTypes (StrictMaybe (..), maybeToStrictMaybe, strictMaybeToMaybe)
6061
import Cardano.Ledger.Coin (Coin (..))
6162
import Cardano.Ledger.Conway.Scripts (PlutusScript (..))
6263
import Cardano.Ledger.Conway.Scripts qualified as Conway
@@ -225,7 +226,9 @@ toLedgerTx = \case
225226
& hashScriptTxWitsL .~ scripts
226227
& datsTxWitsL .~ datums
227228
& rdmrsTxWitsL .~ redeemers
228-
in mkBasicTx body
229+
in mkBasicTx
230+
-- TODO: Test that aux data hash is correctly updated in conversions
231+
(body & auxDataHashTxBodyL .~ maybe SNothing (SJust . hashTxAuxData) auxData)
229232
& isValidTxL .~ toLedgerScriptValidity validity
230233
& auxDataTxL .~ maybeToStrictMaybe auxData
231234
& witsTxL .~ wits

0 commit comments

Comments
 (0)