Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make weak subjectivity check mandatory #7342

Closed
terencechain opened this issue Sep 24, 2020 · 1 comment
Closed

Make weak subjectivity check mandatory #7342

terencechain opened this issue Sep 24, 2020 · 1 comment
Assignees
@farazdagi
Labels
Blocked Blocked by research or external factors Tracking Gotta Catch 'Em All

Comments

@terencechain
Copy link
Member

terencechain commented Sep 24, 2020 •
edited
Loading

Once the weak subjectivity check is well tested and widely adopted in production, we should make weak subjectivity check mandatory in run time.

It's currently not mandatory as anyone can skip the flag and the node will sync pass
@terencechain terencechain added the Tracking Gotta Catch 'Em All label Sep 24, 2020
@terencechain terencechain mentioned this issue Sep 24, 2020
Merged
@rauljordan rauljordan added this to the Diamond milestone Nov 3, 2020
@terencechain
Copy link
Member Author

This is blocked by infra support. Let's see how other clients tackle this

@terencechain terencechain added the Blocked Blocked by research or external factors label Nov 5, 2020
@rauljordan rauljordan removed this from the Diamond milestone Nov 10, 2020
@farazdagi farazdagi self-assigned this Apr 16, 2021
@kasey kasey mentioned this issue Nov 2, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@farazdagi farazdagi

Labels
Blocked Blocked by research or external factors Tracking Gotta Catch 'Em All
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@farazdagi @rauljordan @terencechain