fix: CBOR marshaling for Byron addresses (#822)

agaffney · web-flow · commit 4f0812b8180c · 2025-01-06T17:43:09.000-06:00
Fixes #821
diff --git a/ledger/common/address.go b/ledger/common/address.go
@@ -179,7 +179,11 @@ func (a *Address) UnmarshalCBOR(data []byte) error {
 }
 
 func (a *Address) MarshalCBOR() ([]byte, error) {
-	return cbor.Encode(a.Bytes())
+	addrBytes := a.Bytes()
+	if a.addressType == AddressTypeByron {
+		return addrBytes, nil
+	}
+	return cbor.Encode(addrBytes)
 }
 
 // PaymentAddress returns a new Address with only the payment address portion. This will return nil for anything other than payment and script addresses

Original file line number	Diff line number	Diff line change
`@@ -179,7 +179,11 @@ func (a *Address) UnmarshalCBOR(data []byte) error {`
`179`	`179`	`}`
`180`	`180`
`181`	`181`	`func (a *Address) MarshalCBOR() ([]byte, error) {`
`182`		`- return cbor.Encode(a.Bytes())`
	`182`	`+ addrBytes := a.Bytes()`
	`183`	`+ if a.addressType == AddressTypeByron {`
	`184`	`+ return addrBytes, nil`
	`185`	`+ }`
	`186`	`+ return cbor.Encode(addrBytes)`
`183`	`187`	`}`
`184`	`188`
`185`	`189`	`// PaymentAddress returns a new Address with only the payment address portion. This will return nil for anything other than payment and script addresses`