Fix incompatible DkgPublicKey byte serialization

[piotr-roslaniec]

Type of PR:

• Bugfix

Required reviews:

• 1

What this does:

• Fixes incompatibility of DkgPublicKey serialized to bytes between Python and WASM bindings

Why it's needed:

• Previously added serialization method only affected Python bindings and caused incompatibility with WASM bindings

Notes for reviewers:

• Now downstream effect on nucypher

[codecov-commenter]

Codecov Report

Merging #128 (e516562) into development (ccdc209) will decrease coverage by 0.31%.
The diff coverage is 0.00%.

@@               Coverage Diff               @@
##           development     #128      +/-   ##
===============================================
- Coverage        79.09%   78.78%   -0.31%     
===============================================
  Files               24       24              
  Lines             4850     4869      +19     
===============================================
  Hits              3836     3836              
- Misses            1014     1033      +19     

Impacted Files	Coverage Δ
ferveo/src/api.rs	88.79% <0.00%> (-0.78%)	⬇️
ferveo/src/bindings_python.rs	54.76% <0.00%> (+0.08%)	⬆️
ferveo/src/bindings_wasm.rs	0.00% <0.00%> (ø)
ferveo/src/lib.rs	97.40% <ø> (ø)

[derekpierre]

Only the last 3 commits are relevant - For some reason GH still renders commits from #127 after rebase

It seems that main is ahead of development... development...nucypher:ferveo:main

Is the intention for this PR to merge into development or main?

[piotr-roslaniec]

Is the intention for this PR to merge into development or main?

To merge into development.

I rebase the development branch over main branch. The HEAD of both of these branches should point to the same commit, i.e. before this PR is merged these branches should be the same.

[derekpierre]

You'll need to rebase this PR branch over development again then, and the additional commits should go away.

[piotr-roslaniec]

That worked, thanks!

[KPrasch]

LGTM!