search

keybase / saltpack

a modern crypto messaging format
https://saltpack.org/
BSD 3-Clause "New" or "Revised" License
992 stars 63 forks source link

Dep-inject functions involving randomness #49

Closed akalin-keybase closed 6 years ago

akalin-keybase commented 6 years ago

This is in preparation for adding more hardcoded tests.

Separate out ephemeral key generation into its own interface separate from keys and keyrings.

Make signcryption return an error if there are no receivers.

Better upper bound for receiver count.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.05%) to 84.802% when pulling a573d70c9f95b4baba164b020d2c4da4e262397f on akalin/rand-interface into 0be3d6e4f3eeb36d314fa6325158c6d07870fc9b on master.

akalin-keybase commented 6 years ago

Per our chat, changed this PR to avoid changing the public API. PTAL!

akalin-keybase commented 6 years ago

Reverted my mistake re. max receiver count

oconnor663 commented 6 years ago

LGTM

akalin-keybase commented 6 years ago

Made sure to test vendoring this into client and kbfs to make sure the API really didn't break!