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

Cherry Pick of PR #9676 + #9391 #9704

Merged
merged 3 commits into from Sep 7, 2021
Merged

Cherry Pick of PR #9676 + #9391 #9704

merged 3 commits into from Sep 7, 2021

Conversation

chevdor
Copy link
Contributor

@chevdor chevdor commented Sep 6, 2021

This PR cherry picks the following PRs into the polkadot-v0.9.10 branch for the 0.9.10 release:

It contains what #9707 did, making #9707 no longer required.

@chevdor chevdor 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. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels Sep 6, 2021
* RUSTSEC-2021-0076 bump libsecp256k1

libsecp256k1 allows overflowing signatures
https://rustsec.org/advisories/RUSTSEC-2021-0076

Changes were made to conform to libsecp256k1 version differences.

Closes #9356

* parse_standard_slice() -> parse_overflowing_slice()

* Added v2 host function for ecdsa_verify

* Add feature tag over helpers

* Added ecdsa_verify v2 to test runner

* PR feedback

- Spaces -> tabs
- renamed two helper functions

* Fixed imports after rebasing

* Bump rest of libsecp256k1 (and libp2p)

libp2p also uses libsecp256k1 so it is required to be bumped too, along
with all the version difference changes.

* Add version2 for ecdsa pubkey recovery

* libp2p rebase master fixes

* Fix test panic when non Behaviour event is returned

* Update bin/node/browser-testing/Cargo.toml

* Update primitives/core/src/ecdsa.rs

* Update primitives/core/src/ecdsa.rs

* Update Cargo.lock

Co-authored-by: Bastian Köcher <bkchr@users.noreply.github.com>
@chevdor chevdor force-pushed the wk-0.9.10_pr-9676 branch 2 times, most recently from 9da0f74 to c688e3a Compare September 6, 2021 16:58
@chevdor chevdor mentioned this pull request Sep 7, 2021
@chevdor chevdor changed the title Cherry Pick of PR #9676 Cherry Pick of PR #9676 + #9391 Sep 7, 2021
@chevdor
Copy link
Contributor Author

chevdor commented Sep 7, 2021

The last commit 728e7ff is to prevent having to merge #9541 as well, while having all prost deps set to 0.8.x. There will be no need to backport since #9541 removes the crate entirely.

@chevdor
Copy link
Contributor Author

chevdor commented Sep 7, 2021

bot merge

@ghost
Copy link

ghost commented Sep 7, 2021

Error: When trying to meet the "Project Owners" approval requirements: this pull request does not belong to a project defined in Process.json.

Approval by "Project Owners" is only attempted if other means defined in the criteria for merge are not satisfied first.

The following errors might have affected the outcome of this attempt:

  • 'Benchmarking and Weights' does not match any projects in substrate's Process.json

@chevdor chevdor added this to In Progress 💪 in SDK Node (deprecated) via automation Sep 7, 2021
@chevdor
Copy link
Contributor Author

chevdor commented Sep 7, 2021

bot merge

@ghost
Copy link

ghost commented Sep 7, 2021

Error: Approval criteria was not satisfied.

The following errors might have affected the outcome of this attempt:

  • 'Benchmarking and Weights' does not match any projects in substrate's Process.json

Merge failed. Check out the criteria for merge.

SDK Node (deprecated) automation moved this from In Progress 💪 to Reviewed 😎 Sep 7, 2021
@chevdor
Copy link
Contributor Author

chevdor commented Sep 7, 2021

bot merge

@ghost
Copy link

ghost commented Sep 7, 2021

Merge aborted: Checks failed for 728e7ff

@bkchr bkchr merged commit 56a3f5d into polkadot-v0.9.10 Sep 7, 2021
SDK Node (deprecated) automation moved this from Reviewed 😎 to Done ✔️ Sep 7, 2021
@bkchr bkchr deleted the wk-0.9.10_pr-9676 branch September 7, 2021 10:18
cmichi pushed a commit that referenced this pull request Sep 9, 2021
* RUSTSEC-2021-0076 bump libsecp256k1 (#9391)

* RUSTSEC-2021-0076 bump libsecp256k1

libsecp256k1 allows overflowing signatures
https://rustsec.org/advisories/RUSTSEC-2021-0076

Changes were made to conform to libsecp256k1 version differences.

Closes #9356

* parse_standard_slice() -> parse_overflowing_slice()

* Added v2 host function for ecdsa_verify

* Add feature tag over helpers

* Added ecdsa_verify v2 to test runner

* PR feedback

- Spaces -> tabs
- renamed two helper functions

* Fixed imports after rebasing

* Bump rest of libsecp256k1 (and libp2p)

libp2p also uses libsecp256k1 so it is required to be bumped too, along
with all the version difference changes.

* Add version2 for ecdsa pubkey recovery

* libp2p rebase master fixes

* Fix test panic when non Behaviour event is returned

* Update bin/node/browser-testing/Cargo.toml

* Update primitives/core/src/ecdsa.rs

* Update primitives/core/src/ecdsa.rs

* Update Cargo.lock

Co-authored-by: Bastian Köcher <bkchr@users.noreply.github.com>

* Use coherent prost crate  versions (#9676)

* Bump node-browser-testing deps on prost

Co-authored-by: Trevor Arjeski <72849114+trevor-crypto@users.noreply.github.com>
Co-authored-by: Bastian Köcher <bkchr@users.noreply.github.com>
Co-authored-by: Andreas Doerr <adoerr@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
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
No open projects
SDK Node (deprecated)
  
Done ✔️
Development

Successfully merging this pull request may close these issues.

None yet

4 participants