rust-nostr / negentropy

Rust implementation of the negentropy set-reconcilliation protocol.
MIT License
17 stars 3 forks source link

Changes for protocol version 1 #5

Closed hoytech closed 11 months ago

hoytech commented 11 months ago

Hi Yuki!

I have made some pretty major changes to the negentropy protocol, which is now "version 1" (previous was version 0).

I made the updates to your Rust implementation that bring it to parity with the C++ and JS implementations.

I'm going to be releasing a strfry instance that supports syncing with protocol version 1 very soon.

Happy to answer any questions, and please forgive any Rust-related issues since I'm not very experienced in this language.

I have not yet tested any of the bindings to other languages.

Cheers, Doug

yukibtc commented 11 months ago

Hi Doug, thanks for the update!

I made some cleanups and fixes in PR #6. Can you confirm that harness test pass?

If you are interested to test bindings (for now I packaged only python, I hope to package also Kotlin, Swift and Ruby soon) these are the steps:

hoytech commented 11 months ago

Thanks! There was a small issue surfaced by the harness tests, I've opened PR #7 to fix that.

I will try to find some time to look at the Python bindings soon.