Skip to content

types: update tx hash to use SHA256 and no amino #1547

@ebuchman

Description

@ebuchman

Currently the tx hash is computed as RIPEMD160(amino_encode(tx))

This has the effect of just prefixing the varint length before hashing.

We should avoid doing this, for simplicity (ie. just the hash of the underlying tx - it would already be encoded with something like amino at the app level).

Also we should move to SHA256[:20]

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions