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

Time sync ntp #89

Merged
merged 17 commits into from
Feb 11, 2018
Merged

Time sync ntp #89

merged 17 commits into from
Feb 11, 2018

Conversation

y0sher
Copy link
Contributor

@y0sher y0sher commented Feb 8, 2018

Basic implementation of querying multiple NTP servers and calculate an average drift.
block the user at app level startup if the drift is too big.

chcek the drift every half an hour and shutdown if drift is too big.

future TODO:

  • add some security layer to check ntp packet isn't compromised ?

@avive
Copy link
Contributor

avive commented Feb 9, 2018

It is super exciting to see the great progress on this PR !

@y0sher y0sher changed the title WIP - Time sync ntp Time sync ntp Feb 11, 2018
@y0sher y0sher requested a review from avive February 11, 2018 15:13
@y0sher y0sher mentioned this pull request Feb 11, 2018
@@ -0,0 +1,148 @@
package timesync
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider moving package out from package p2p to root.

@avive avive merged commit 1aefa73 into spacemeshos:develop Feb 11, 2018
@y0sher y0sher deleted the time-sync-ntp branch April 18, 2018 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants