search

crate-crypto / rust-verkle

Apache License 2.0
119 stars 41 forks source link

fixed unsafe code for absent keys #56

Closed steph-rs closed 2 years ago

steph-rs commented 2 years ago

old version of code threw panic with absent keys

steph-rs commented 2 years ago

i used old version without banderwagon, but I didn't find any active branches with that version

kevaundray commented 2 years ago

Hi, the master branch should be using banderwagon. We no longer support bandersnatch

kevaundray commented 2 years ago

Change looks good either way though :)

steph-rs commented 2 years ago

Yeah, i understand. But currently i'm trying to work with condrieu test-net, so I have some issues with decoding proof with banderwagon :(

steph-rs commented 2 years ago

Oh, sorry. Last commit is excess

kevaundray commented 2 years ago

Yeah, i understand. But currently i'm trying to work with condrieu test-net, so I have some issues with decoding proof with banderwagon :(

The proof decoding is because Rust and Go disagree on the proof format, we have a few issues opened up for it