Skip to content

Add author_generateOwnershipProof RPC endpoint#11493

Draft
sigurpol wants to merge 4 commits intomasterfrom
sigurpol-get-proof
Draft

Add author_generateOwnershipProof RPC endpoint#11493
sigurpol wants to merge 4 commits intomasterfrom
sigurpol-get-proof

Conversation

@sigurpol
Copy link
Contributor

Allow validators to generate an ownership proof for existing session keys without rotating them. This enables validators who already have keys in their keystore to reuse existing keys, generate a valid proof via the new endpoint and then call set_keys on Asset Hub without the need to generate new keys via author_rotateKeysWithOwner RPC endpoint.

Driven-by:

  • Bump SessionKeys runtime API to v3

Allow validators to generate an ownership proof for existing session
keys without rotating them. This enables validators who already have
keys in their keystore to submit set_keys on Asset Hub without
unnecessary key regeneration.

Changes:
- Bump SessionKeys runtime API to v3 with new
  generate_ownership_proof method
- Add author_generateOwnershipProof RPC endpoint
@sigurpol sigurpol requested a review from a team as a code owner March 25, 2026 11:35
@sigurpol
Copy link
Contributor Author

/cmd prdoc --audience runtime_dev --bump patch

@sigurpol sigurpol added the T3-RPC_API This PR/Issue is related to RPC APIs. label Mar 25, 2026
@bkchr
Copy link
Member

bkchr commented Mar 25, 2026

@sigurpol why we need this?

@sigurpol sigurpol marked this pull request as draft March 25, 2026 15:15
@sigurpol
Copy link
Contributor Author

sigurpol commented Mar 25, 2026

@sigurpol why we need this?

I have strong doubts myself now too about this PR 😄

The use case would be the one where for a validator, keys are generated by a separate service one by one and it doesn't get the proof along with them.
We would first call author_insertKeys() for each key and then this new RPC endpoint to get associated proofs and then finally we would call set_keys().

I am currently thinking what alternative / solution we can offer to cover this use case.

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

Labels

T3-RPC_API This PR/Issue is related to RPC APIs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants