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

Move `hasPreimage` / `getPreimage` from Validator API to Full Node API #566

Closed
bpalaggi opened this issue Feb 20, 2020 · 1 comment
Closed

Move `hasPreimage` / `getPreimage` from Validator API to Full Node API #566

bpalaggi opened this issue Feb 20, 2020 · 1 comment

Comments

@bpalaggi
Copy link

@bpalaggi bpalaggi commented Feb 20, 2020

Related: #567

Currently hasPreimage (to be changed to getPreimage) is part of Validator's API.
It should be part of the full node API, as preimage need to be known to full node in order to verify signature (and it also increases data availability for them).

@bpalaggi bpalaggi added this to the 2. Validator milestone Feb 20, 2020
@Geod24 Geod24 changed the title Move 'get preimage' to Full Node Move `hasPreimage` / `getPreimage` from Validator API to Full Node API Feb 20, 2020
@Geod24

This comment has been minimized.

Copy link
Member

@Geod24 Geod24 commented Feb 20, 2020

Edited

@Geod24 Geod24 added this to To do in Sprint #12 (2020-02-25 to 2020-03-9) via automation Feb 25, 2020
@bpalaggi bpalaggi assigned bpalaggi and linked0 and unassigned bpalaggi Feb 25, 2020
@linked0 linked0 moved this from To do to In progress (Max 5) in Sprint #12 (2020-02-25 to 2020-03-9) Mar 4, 2020
@linked0 linked0 moved this from In progress (Max 5) to Review/Testing (Max 2) in Sprint #12 (2020-02-25 to 2020-03-9) Mar 4, 2020
@Geod24 Geod24 closed this in #617 Mar 6, 2020
Sprint #12 (2020-02-25 to 2020-03-9) automation moved this from Review/Testing (Max 2) to Done Mar 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants
You can’t perform that action at this time.