Skip to content

RFC 0023 - Sign VRF - #301

Open
valentunn wants to merge 4 commits into
mainfrom
rfc/0023
Open

RFC 0023 - Sign VRF#301
valentunn wants to merge 4 commits into
mainfrom
rfc/0023

Conversation

@valentunn

@valentunn valentunn commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

@valentunn
valentunn requested review from a team July 21, 2026 10:26
Comment on lines +25 to +27
the host signs locally. The motivating consumer is the People Chain lottery
ticket a participant submits while **not yet a people-set member**; members use
the bandersnatch ring-VRF path instead (`create_account_proof`, RFC-0004).

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can't you just use create_account_proof if you are part of people lite?
if so why would I use that instead ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a question to runtime cc @georgepisaltu

Overall, this question seems to be not particularly related to this RFC - whether or nor lite people is applicable instead of shnorr vrf doesn't affect the shape of the sign vrf API itself.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am just questioning the use case inside the TrUAPI where you can either use lite or full people bandersnatch ring-VRF proof. You framed it as

participant submits while not yet a people-set member; members use
the bandersnatch ring-VRF path instead (create_account_proof, RFC-0004)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tbh I am not sure I got the question, in particular which part of this RFC is concerning / what is the alternative?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

my question is simple do we need this if one can just lite people proof instead.
if the answer is yes let's ignore my noise and resolve this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants