ed25519-dalek icon indicating copy to clipboard operation
ed25519-dalek copied to clipboard

Attempt at implementing streaming verification

Open Monadic-Cat opened this issue 1 year ago • 2 comments

As requested here, this PR contains my port of PR #196 to use the hazmat module instead of implementing any cryptographic stuff itself.

I haven't copied over the added test yet, but I have confirmed basic functionality by test using it in a demo application I have lying around.

Monadic-Cat avatar May 16 '23 20:05 Monadic-Cat

Thanks for pushing this up. It's interesting you managed to make it work but I guess I misjudged whether this would be a good fit. Feels like the wrong abstraction level.

Perhaps VerifyingKey::recompute_R can be subdivided into some smaller functions (that is to say, preserve recompute_R, but have it call some other functions) which are more amenable to this streaming verification use case.

e.g. recompute_R_from_k?

tarcieri avatar May 16 '23 22:05 tarcieri

Looks like @robjtede updated #196 so perhaps we should continue the discussion there

tarcieri avatar May 16 '23 23:05 tarcieri