vocdoni / saas-backend / 29410417712

62%

main: 62%

Build Type

Pull #579

github

Committed by lucasmenendez

Commit Message

fix(processes): serialize org address as internal.HexBytes (bare hex)

The /processes API typed orgAddress as common.Address, which JSON-marshals to a
0x-prefixed EIP-55-checksummed string — inconsistent with upstreamId (and the
other hex ids in these types), which use internal.HexBytes (bare lowercase hex).
Switch CreateVotingProcessRequest.OrgAddress and VotingProcessResponse.OrgAddress
to internal.HexBytes for a consistent API.

- Response: VotingProcessResponseFromDB converts the db common.Address via .Bytes().
- Request: createVotingProcessHandler validates len == common.AddressLength
  (HexBytes, unlike common.Address, doesn't enforce a 20-byte length on decode)
  and converts once to common.Address for the db/auth/subscription calls.
- Inputs stay compatible: HexBytes.UnmarshalJSON strips an optional 0x prefix, so
  clients may still send "0x…". Only the response format changes to bare hex.

Scope is limited to what #571 introduced; legacy /process and census types keep
common.Address.

Pull Request Pull Request #579: fix(processes): serialize org address as internal.HexBytes (bare hex)

Coverage Stats

8 of 9 new or added lines in 2 files covered. (88.89%)

56 existing lines in 1 file now uncovered.

11494 of 18489 relevant lines covered (62.17%)

44.71 hits per line