IntersectMBO / cardano-node

The core component that is used to participate in a Cardano decentralised blockchain.
https://cardano.org
Apache License 2.0
3.06k stars 721 forks source link

[IMPROVEMENT] - Helpful error message when assetname is too long #4619

Closed cardano-tools-nft closed 1 year ago

cardano-tools-nft commented 2 years ago

Hi

calculate-min-required-utxo has an problem with my particular assetname, cant see whats wrong with it:

bash-5.1# cardano-cli --version
cardano-cli 1.35.3 - linux-x86_64 - ghc-8.10
git rev 0000000000000000000000000000000000000000
bash-5.1# cardano-cli transaction calculate-min-required-utxo --protocol-params-file protocol.json --babbage-era --tx-out "addr1q9h2kw7neeh26cmhspye2gyvhca6qaqxakp7qcp3uydzphqu62gur507vnszr24dyt2z6lrmpmrufs6jjdhc2vy04hzqyuuw8z+1 ee659e0a80e8a9815afe8aa3906741c34d234923f4da0543ec1628d4.5468654D616E6472696C6C7A576F726C6443757045646974696F6E2333353023373731"
option --tx-out:
unexpected end of input
expecting hexadecimal digit
AssetName deserisalisation failed: Failed to deserialise "5468654D616E6472696C6C7A576F726C6443757045646974696F6E2333353023373731"
cardano-tools-nft commented 2 years ago

I see assetnames can be maximum 32 bytes, but a helpful error message would be nice.

github-actions[bot] commented 1 year 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.