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

NTP Management Modification #158

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

NTP Management Modification #158

wants to merge 1 commit into from

Conversation

heaton84
Copy link

This is a nice library, almost exactly what I was looking for!

I made a couple of changes to suit my needs, namely:

  1. I added method to read variables from an NTP server via control packets. My angle is to store a timezone variable in the NTP server, and have all my clocks get the timezone data from that server. Example usage:
Timezone localTime;

bool updateTimezone()
{
  String r;
  if (readNTPVariable("TZ", r))
  {
    localTime.setPosix(r.c_str());
    
    return true;
  }
  return false;
}
  1. I also modified queryNTP to respect the server poll if our current poll is set too quickly. This is a courtesy to NTP server owners.

Added method to read variables from an NTP server.

Also modified NTP query to respect the server poll if our current poll is set too quickly.
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.

1 participant