IntersectMBO / cardano-cli

This repository contains sources for the command-line interface (CLI) tool for interacting with the Cardano blockchain.
Apache License 2.0
44 stars 15 forks source link

[MAINTENANCE] - Remove usages of `SafeHash` #952

Open palas opened 1 month ago

palas commented 1 month ago

What

Replace all usages of SafeHash with unverified Hash unless in cases where it is possible to track the source of the hash. If somewhere this is unavoidable, wait till the last second before unsafely converting to SafeHash.

Why

SafeHash is meant to encode that the hash has a pre-image, but this is not enforceable most of the time in practice, since both cardano-cli and cardano-api are often provided with the hash only, not with the pre-image.

Acceptance Criteria

github-actions[bot] commented 4 days ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.