Skip to content

replace panic for usupported liquid with error #151

@zoedberg

Description

@zoedberg

I think we should replace the panics saying "Liquid is not yet supported" (e.g. https://github.com/RGB-WG/rgb/blob/master/src/resolvers/electrum.rs#L168 and https://github.com/RGB-WG/rgb/blob/master/src/resolvers/esplora_blocking.rs#L110) with a dedicated error, because otherwise the validation of a consignment containing a liquid anchor can panic. I know it should be impossible creating such consignment with current crates, but a malicious user could edit the consignment using YAML import/export feature and upload it to the proxy (or another consignment transport medium) making a wallet panic.

@dr-orlovsky let me know if you want me to open a PR or if you prefer to do it

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

Status

Done

Relationships

None yet

Development

No branches or pull requests

Issue actions