Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

fix(staking miner): use StorageKey in rpc call getStorage #4231

Merged

Conversation

niklasad1
Copy link
Member

Use StorageKey in the rpc call state_getStorage instead of [u8;32], the reason is that StorageKey is serialized as hex string and rejects types that is serialized as array.

Use `StorageKey` in the rpc call `getStorage` instead of `[u8;32]`
@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Nov 5, 2021
@niklasad1 niklasad1 changed the title fix(staking miner): use StorageKey in getStorage fix(staking miner): use StorageKey in rpc call getStorage Nov 5, 2021
@niklasad1 niklasad1 added B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Nov 5, 2021
@emostov
Copy link
Contributor

emostov commented Nov 6, 2021

bot merge

@paritytech-processbot paritytech-processbot bot merged commit d985794 into master Nov 6, 2021
@paritytech-processbot paritytech-processbot bot deleted the na-fix-staking-miner-get_storage-call branch November 6, 2021 07:16
Lldenaurois pushed a commit that referenced this pull request Nov 6, 2021
Use `StorageKey` in the rpc call `getStorage` instead of `[u8;32]`
ordian added a commit that referenced this pull request Nov 12, 2021
* master: (71 commits)
  Remove light client companion (#4191)
  Enable full use of pallet-bags-list in westend and kusama runtimes (#4195)
  collator-protocol: do not connect to the next group (#4261)
  Per subsystem CPU usage tracking (#4239)
  CI: chore (#3957)
  Companion – Update jsonrpsee to 0.4.1 (#4256)
  Add more XCM tracing (#4211)
  Update dependencies for latest substrate master (#4258)
  Bump mick-jaeger from 0.1.4 to 0.1.6 (#4249)
  Bump dlmalloc from 0.2.2 to 0.2.3 (#4250)
  Bump libc from 0.2.106 to 0.2.107 (#4235)
  Bump paste from 1.0.5 to 1.0.6 (#4244)
  Bump serde_json from 1.0.68 to 1.0.69 (#4236)
  Update `wasmtime` and related dependencies (companion for Substrate#10149) (#4210)
  update cargo lock to unbreak dep of a dep (#4245)
  Increase maximum chunk size to adjust for small networks. (#4220)
  availability recovery type name clarifications (#4203)
  Update `bridge/` codeowners (#4222)
  fix(staking miner): use `StorageKey` in getStorage (#4231)
  Change path for the tests to master (#4223)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants