Repository navigation
std::error::Error implementation for hex::Error #258
Description
It seems that elements::hex::Error is missing std::error::Error implementation, which makes it difficult to work with dynamic-error crates (like eyre or anyhow), as there default ::from method requires that.
For example,
let bytes_res = Vec::<u8>::from_hex(&raw_tx).map_err(eyre::Report::from);doesn't compile, as eyre::Report::from requires T: std::error::Error, but:
let bytes_res = Vec::<u8>::from_hex(&raw_tx).map_err(|e| eyre::eyre!(e));works, as it does some black-magic under the hood with the value. So still from implementation is preferable
Activity
apoelstra commentedon Oct 13, 2025
We should really just drop this module entirely in favor of hex-conservative.
Velnbur commentedon Oct 29, 2025
Any updates for this issue? I may do migration to
hex-conservativein my free timeapoelstra commentedon Oct 29, 2025
No updates. If you want to do it then patches welcome!
Velnbur commentedon Nov 2, 2025
@apoelstra Added #259 check it out
Merge ElementsProject#259: Replace custom hex parsing with hex-conser…