Skip to content

staking-async-rc-client: add proof parameter to set_keys #11255

Merged
sigurpol merged 5 commits intostable2512from
sigurpol-staking-async-rc-client-proof-in-set-keys-no-op-validation
Mar 4, 2026
Merged

staking-async-rc-client: add proof parameter to set_keys #11255
sigurpol merged 5 commits intostable2512from
sigurpol-staking-async-rc-client-proof-in-set-keys-no-op-validation

Conversation

@sigurpol
Copy link
Copy Markdown
Contributor

@sigurpol sigurpol commented Mar 4, 2026

Add proof: Vec<u8> to the set_keys extrinsic for forward compatibility with PR #1739, which introduces real ownership proof validation.
Proof validation is a no-op until we backport PR #1739.

Note: I have added validate:false for pallet-staking-async-rc-client. The set_keys extrinsic has never been deployed on Polkadot/Kusama AH yet, it will be deployed just with release 2.1.0 the first time so it's a brand new API release-wise introduced with the backport of #10666.

Add `proof: Vec<u8>` to the `set_keys` extrinsic for forward compatibility
with PR #1739, which introduces real ownership proof validation.

Proof validation is a no-op until we backport PR #1739.
@sigurpol sigurpol requested a review from a team as a code owner March 4, 2026 10:48
…g-async-rc-client-proof-in-set-keys-no-op-validation
@sigurpol sigurpol force-pushed the sigurpol-staking-async-rc-client-proof-in-set-keys-no-op-validation branch from 8a8c624 to 58c2ec8 Compare March 4, 2026 10:59
@sigurpol
Copy link
Copy Markdown
Contributor Author

sigurpol commented Mar 4, 2026

/cmd prdoc --audience runtime_dev --bump minor

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 4, 2026

This pull request is amending an existing release. Please proceed with extreme caution,
as to not impact downstream teams that rely on the stability of it. Some things to consider:

  • Backports are only for 'patch' or 'minor' changes. No 'major' or other breaking change.
  • Should be a legit fix for some bug, not adding tons of new features.
  • Must either be already audited or not need an audit.
Emergency Bypass

If you really need to bypass this check: add validate: false to each crate
in the Prdoc where a breaking change is introduced. This will release a new major
version of that crate and all its reverse dependencies and basically break the release.

@sigurpol sigurpol added the T2-pallets This PR/Issue is related to a particular pallet. label Mar 4, 2026
@sigurpol sigurpol enabled auto-merge (squash) March 4, 2026 13:19
@sigurpol sigurpol merged commit 3dd5b89 into stable2512 Mar 4, 2026
264 of 275 checks passed
@sigurpol sigurpol deleted the sigurpol-staking-async-rc-client-proof-in-set-keys-no-op-validation branch March 4, 2026 14:09
fellowship-merge-bot bot added a commit to polkadot-fellows/runtimes that referenced this pull request Mar 7, 2026
…of of ownership in staking-async-rc-client (#1101)

This PR
- bumps pallet-society to introduce `kick_member` as per
paritytech/polkadot-sdk#11280
- adds support for proof as input parameters (but no real validation
since paritytech/polkadot-sdk#1739 has not been
backported to `stable2512` ) as per
paritytech/polkadot-sdk#11255
- adds support for storage keys deposit on staking-async-rc-client as
per paritytech/polkadot-sdk#11242 (10 DOT for
Polkadot AssetHub, 1 KSM for Kusama AssetHub)

## TODO 

- [ ] (if there is time) re-run benchmark for pallet-society on
asset-hub-kusama. Not critical since kick_member is meant to be used
once in a lifetime but nice to have
- [x]  CHANGELOG

---------

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Co-authored-by: fellowship-merge-bot[bot] <151052383+fellowship-merge-bot[bot]@users.noreply.github.com>
arturgontijo pushed a commit to moonbeam-foundation/polkadot-sdk that referenced this pull request Apr 1, 2026
…11255)

Add `proof: Vec<u8>` to the `set_keys` extrinsic for forward
compatibility with PR paritytech#1739, which introduces real ownership proof
validation.
Proof validation is a no-op until we backport PR paritytech#1739.

Note: I have added `validate:false` for pallet-staking-async-rc-client.
The `set_keys` extrinsic has never been deployed on Polkadot/Kusama AH
yet, it will be deployed just with release `2.1.0` the first time so
it's a brand new API release-wise introduced with the backport of
paritytech#10666.

---------

Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T2-pallets This PR/Issue is related to a particular pallet.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants