Navigation Menu

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.

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

Implement Secure Sync #1508

Closed
1 of 2 tasks
nazar-pc opened this issue Jun 6, 2023 · 2 comments · Fixed by #1586
Closed
1 of 2 tasks

Implement Secure Sync #1508

nazar-pc opened this issue Jun 6, 2023 · 2 comments · Fixed by #1586
Assignees
Labels
core Related to core protocol, may affect fundamentals epic networking Subspace networking (DSN) node Node (service library/node app)

Comments

@nazar-pc
Copy link
Member

nazar-pc commented Jun 6, 2023

Once we have Defined initial secure sync spec, we need to implement it.

More changes might be necessary if written spec has more updated on top of what we have discussed previously.

Background information: https://www.notion.so/subspacelabs/Define-Secure-Consensus-Chain-Sync-Protocol-bba1a73e8ba949cea413eb7dc5281173

@nazar-pc nazar-pc added networking Subspace networking (DSN) node Node (service library/node app) core Related to core protocol, may affect fundamentals labels Jun 6, 2023
@nazar-pc nazar-pc added this to the Secure Sync from DSN milestone Jun 6, 2023
@nazar-pc nazar-pc self-assigned this Jun 6, 2023
@nazar-pc
Copy link
Member Author

nazar-pc commented Jun 7, 2023

@dariolina do you remember what do we mean here exactly? Node will already get stuck if it syncs onto the wrong chain and unable to move forward. If it is on the wrong fork however, it will not be able to objectively tell that it is on the wrong chain, at least not by default.

Looks like it is a non-actionable task so far.

@dariolina
Copy link
Member

IIRC we wanted if the node later discovers a heavier chain than its own to present an error to the user, so they wipe and start over. If you would like to refresh your memory we have discussed it in this session at ~2:29:00. An the whole sync discussion started around 02:00:00

@nazar-pc nazar-pc mentioned this issue Jun 20, 2023
1 task
This was referenced Jun 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Related to core protocol, may affect fundamentals epic networking Subspace networking (DSN) node Node (service library/node app)
Projects
Development

Successfully merging a pull request may close this issue.

3 participants