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

A simple check to reject obviously wrong validation code binaries #1989

Merged
4 commits merged into from
Nov 19, 2020
Merged

A simple check to reject obviously wrong validation code binaries #1989

4 commits merged into from
Nov 19, 2020

Conversation

pepyakin
Copy link
Contributor

Would have saved us a bit of headache

@pepyakin pepyakin 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. labels Nov 19, 2020
ordian
ordian reviewed Nov 19, 2020
View reviewed changes
runtime/common/src/paras_registrar.rs Outdated Show resolved Hide resolved
runtime/common/src/paras_registrar.rs Outdated Show resolved Hide resolved
runtime/common/src/paras_registrar.rs Outdated Show resolved Hide resolved
@pepyakin
Copy link
Contributor Author

bot merge

@ghost
Copy link

ghost commented Nov 19, 2020

Trying merge.

@ghost ghost merged commit de70336 into master Nov 19, 2020
@ghost ghost deleted the ser-definitely-not-wasm branch November 19, 2020 19:22
ordian added a commit that referenced this pull request Nov 19, 2020
@ordian
Merge branch 'master' into ao-session-info
4599423 
* master:
  A simple check to reject obviously wrong validation code binaries (#1989)
  fix approval keytypes doc (#1988)
  add max_pov_size to runtime config and PersistedValidationData (#1984)
  Bump env_logger from 0.8.1 to 0.8.2 (#1981)
  Bump pin-project from 1.0.1 to 1.0.2 (#1982)
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ordian ordian ordian approved these changes

Assignees
No one assigned
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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pepyakin @ordian