Skip to content

Commit ac82e17

Browse files
authored
guardiand: fix set message fee serialization (wormhole-foundation#4397)
1 parent 40ddcfe commit ac82e17

3 files changed

Lines changed: 4 additions & 5 deletions

File tree

node/cmd/guardiand/adminverify.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,6 @@ func runGovernanceVAAVerify(cmd *cobra.Command, args []string) {
5757
if err != nil {
5858
log.Fatal(err.Error())
5959
}
60-
log.Printf("VAA with digest %s: %+v\n", digest, debugStr)
60+
log.Printf("VAA with digest %x: %+v\n", digest, debugStr)
6161
}
6262
}

sdk/vaa/payloads.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -503,7 +503,6 @@ func (r BodyWormholeRelayerSetDefaultDeliveryProvider) Serialize() ([]byte, erro
503503

504504
func (r BodyCoreBridgeSetMessageFee) Serialize() ([]byte, error) {
505505
payload := &bytes.Buffer{}
506-
MustWrite(payload, binary.BigEndian, r.ChainID)
507506
MustWrite(payload, binary.BigEndian, r.MessageFee)
508507
return serializeBridgeGovernanceVaa(CoreModuleStr, ActionCoreSetMessageFee, r.ChainID, payload.Bytes())
509508
}

sdk/vaa/payloads_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -457,10 +457,10 @@ func TestBodyRecoverChainIdModuleTooLong(t *testing.T) {
457457
}
458458

459459
func TestBodyCoreBridgeSetMessageFeeSerialize(t *testing.T) {
460-
expected := "00000000000000000000000000000000000000000000000000000000436f72650300040004000000000000007b"
460+
expected := "00000000000000000000000000000000000000000000000000000000436f72650304560000000000000123"
461461
bodyCoreBridgeSetMessageFee := BodyCoreBridgeSetMessageFee{
462-
ChainID: 4,
463-
MessageFee: uint64(123),
462+
ChainID: 0x456,
463+
MessageFee: uint64(0x123),
464464
}
465465
buf, err := bodyCoreBridgeSetMessageFee.Serialize()
466466
require.NoError(t, err)

0 commit comments

Comments
 (0)