penumbra icon indicating copy to clipboard operation
penumbra copied to clipboard

Error message in `pcli q shielded-pool nullifier` points to a bug

Open hdevalence opened this issue 2 years ago • 1 comments

Describe the bug

~/c/p/penumbra (main|…) $ cargo run --release --bin pcli --features "nct-divergence-check" -- -n 127.0.0.1 q shielded-pool nullifier d9f51a0170abccfd5f132f9989765354c46506725058ad90db40522263eb3c14
    Finished release [optimized] target(s) in 0.21s
     Running `target/release/pcli -n 127.0.0.1 q shielded-pool nullifier d9f51a0170abccfd5f132f9989765354c46506725058ad90db40522263eb3c14`
error: Invalid value "d9f51a0170abccfd5f132f9989765354c46506725058ad90db40522263eb3c14" for '<NULLIFIER>': Invalid Decaf377 encoding

For more information try --help

This error message points at a problem, because the nullifier isn't a Decaf377 point, it's an Fq element. Probably this is just an error message issue?

hdevalence avatar Jun 21 '22 18:06 hdevalence

This is just an error message issue, because the FieldExt trait uses the EncodingError type from decaf377.

plaidfinch avatar Jun 21 '22 20:06 plaidfinch

Closing since this is just an issue with the error message

zbuc avatar Mar 24 '23 17:03 zbuc