Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: expose roots from zerokit #105

Open
weboko opened this issue Mar 25, 2024 · 1 comment
Open

feat: expose roots from zerokit #105

weboko opened this issue Mar 25, 2024 · 1 comment
Labels
enhancement New feature or request track:rln RLN Track (Secure Messaging/Applied ZK), e.g. relay and applications

Comments

@weboko
Copy link
Contributor

weboko commented Mar 25, 2024

Problem

@waku/rln maintains its roots here by essentially relying on zerokit to give it current root here.

These roots lately are used to verify a proof - which is done again inside of zerokit here.

Solution

Expose from wasm following API:

  • getRoots - gives requested amount of latest roots (default to 5);
  • remove a need to pass roots to verify as they are already inside of zerokit;

getRoots still can be exposed as might be used later or by a consumer.

@weboko weboko added the track:rln RLN Track (Secure Messaging/Applied ZK), e.g. relay and applications label Mar 25, 2024
@weboko weboko added this to Waku Mar 25, 2024
@weboko weboko moved this to Triage in Waku Mar 25, 2024
@weboko weboko added the enhancement New feature or request label Mar 27, 2024
@weboko weboko moved this from Triage to Icebox in Waku Apr 24, 2024
@weboko
Copy link
Contributor Author

weboko commented Apr 24, 2024

Ice boxing for now. Re-prioritizing after approach to RLN is clear in light clients.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request track:rln RLN Track (Secure Messaging/Applied ZK), e.g. relay and applications
Projects
Status: Icebox
Development

No branches or pull requests

1 participant