-
Notifications
You must be signed in to change notification settings - Fork 8
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
Unable to calculate offset #6
Comments
To me it looks like |
Not that I could see! |
That certainly seems reasonable. |
marcbowes
added a commit
to marcbowes/net-ntp
that referenced
this issue
Apr 15, 2014
The test is quite hard if your clock is in sync. I'm not sure the right way to actually do this because we're using a pool of NTP servers and ntpdate and Net::NTP could hit different servers. I think this is good enough for now to verify that things actually work and return reasonable results. closes zencoder#6
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
According to RFC 958, to calculate your offset, you need to use run a calculation that includes the timestamp of when the client received the packet.
Right now, this timestamp is being set in the constructor of
Response
- seenet-ntp/lib/net/ntp/ntp.rb
Line 103 in 234d8a1
Would it be reasonable to:
The text was updated successfully, but these errors were encountered: