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

Require crypto scheme for insert-key #9909

Merged
2 commits merged into from
Oct 4, 2021
Merged

Require crypto scheme for insert-key #9909

2 commits merged into from
Oct 4, 2021

Conversation

bkchr
Copy link
Member

@bkchr bkchr commented Oct 1, 2021

We should not "guess" the scheme as this can depend on the
implementation of the runtime etc.

Fixes: #9888

We should not "guess" the scheme as this can depend on the
implementation of the runtime etc.
@bkchr bkchr added 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. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels Oct 1, 2021
@bkchr bkchr requested a review from nuke-web3 October 1, 2021 11:31
@bkchr
Copy link
Member Author

bkchr commented Oct 4, 2021

bot merge

@ghost
Copy link

ghost commented Oct 4, 2021

Trying merge.

@ghost
Copy link

ghost commented Oct 4, 2021

Bot will approve on the behalf of @bkchr, since they are a team lead, in an attempt to reach the minimum approval count

@ghost ghost merged commit 1d356ce into master Oct 4, 2021
@ghost ghost deleted the bkchr-require-scheme branch October 4, 2021 20:55
ordian added a commit that referenced this pull request Oct 5, 2021
* master: (125 commits)
  Update multiple dependencies (#9936)
  Speed up timestamp generation when logging (#9933)
  First word should be Substrate not Polkadot (#9935)
  Improved file not found error message (#9931)
  don't read events in elections anymore. (#9898)
  Remove incorrect sanity check (#9924)
  Require crypto scheme for `insert-key` (#9909)
  chore: refresh of the substrate_builder image (#9808)
  Introduce block authorship soft deadline (#9663)
  Rework Transaction Priority calculation (#9834)
  Do not propagate host RUSTFLAGS when checking for WASM toolchain (#9926)
  Small quoting comment fix (#9927)
  add clippy to CI (#9694)
  Ensure BeforeBestBlockBy voting rule accounts for base (#9920)
  rm `.maintain` lock (#9919)
  Downstream `node-template` pull (#9915)
  Implement core::fmt::Debug for BoundedVec (#9914)
  Quickly skip invalid transactions during block authorship. (#9789)
  Add SS58 prefix for Automata (#9805)
  Clean up sc-peerset (#9806)
  ...
This pull request was closed.
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. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sc_cli key insert gives wrong name to _gran_ key file created in local keystorage (BREAKS blocks finalization)
2 participants