@@ -2,14 +2,14 @@

  Background Radiation Monitor - Web Server

  A simple web server that makes available to clients over the Internet

- readings from a MightyOhm Geiger counter. The MightyOhm is connected to

- an Arduino Uno with attached Ethernet shield.  This software module

+ readings from a MightyOhm Geiger counter. The MightyOhm is connected

+ to an Arduino Uno with attached Ethernet shield.  This software module

  runs on the Arduino Uno an embedded HTTP server by which Internet

  applications can query the MightyOhm for Geiger counter readings.

  Also, this software runs a Network Time Protocol (NTP) client, that

  periodically synchronizes the local system clock to network time.

- Included is a simple command line interface that may be used to change the

- network interface IP address, NTP server address, or configure a

+ Included is a simple command line interface that may be used to change

+ the network interface IP address, NTP server address, or configure a

  verbose output mode.

  Copyright 2018 Jeff Owrey

@@ -75,6 +75,13 @@

          NTP address set to "pool.ntp.org" per ntp.org request to use

          (in order to facilitate load balancing) the fully qualified

          domain name instead of individual server IP addresses.

+   * v18 released 01 Nov 2019 by J L Owrey

+       - fixed a bug in NTP time synchronization whereby the network time

+         synchronization would only occur during boot up.  Thereafter NTP 

+         time synchronization would fail to happen, resulting in a large 

+         amount of clock drift.

+

+12345678901234567890123456789012345678901234567890123456789012345678901234567890         

 */

 /***  PREPROCESSOR DEFINES  ***/

@@ -85,8 +92,8 @@

  Define the header and version number displayed at startup

  and also by the 'view settings' command.

 */

-#define STARTUP_HEADER "\n\rRadmon v1.7 (c) 2019\n"

-#define RADMON_VERSION "v1.7"

+#define STARTUP_HEADER "\n\rRadmon v1.8 (c) 2019"

+#define RADMON_VERSION "v1.8"

 /*

  The following define sets the MAC address of the device.  This

  address is a permanent attribute of the device's Ethernet interface,

@@ -95,7 +102,7 @@

  specific instance of the Ethernet shield.  This MAC address should

  be shown on a label affixed to the device housing.

 */

-#define ETHERNET_MAC_ADDRESS 0xNN, 0xNN, 0xNN, 0xNN, 0xNN, 0xNN

+#define ETHERNET_MAC_ADDRESS 0x90, 0xA2, 0xDA, 0x0D, 0x84, 0xF6

 /*

  The following defines an APIPA default address in the event that

  DHCP mode is ON and a DHCP address cannot be obtained.

@@ -106,7 +113,7 @@

  out over the device's USB port.  This heartbeat consists of a serial

  data string containing the current radiation reading and GM time.

 */

-#define SERIAL_UPDATE_INTERVAL 5000  //milli-seconds

+#define SERIAL_UPDATE_INTERVAL 1000  //milli-seconds

 /*

  The following define sets the port number the HTTP service will use to

  listen for requests from Internet clients.  Normally HTTP requests use

@@ -116,18 +123,19 @@

 /*

  The following defines are for configuring a local NTP client

  for synchronizing the local system clock to network time.

- Note that the default setting is the IP address of the following

- time server:

-              time-c-b.nist.gov

+ Note that the ntp server address should be sent to the local

+ server pool of the country where the radmon will be used.  See

+ the web site 'ntp.org' for details. Users in the USA should set

+ the ntp server to 'us.pool.ntp.org'.

 */

 #define DEFAULT_NTP_SERVER_ADDR "pool.ntp.org"

 #define NTP_PORT 8888

-#define NTP_PACKET_SIZE 48 // NTP time stamp is in the first 48 bytes of the message

+#define NTP_PACKET_SIZE 48 // NTP time in the first 48 bytes of the message

 /*

  The following defines how often the system clock gets synchronized

  to network time.

 */

-#define NET_SYNCH_INTERVAL 21600 //number in seconds - 4 times a day

+#define NTP_SYNCH_INTERVAL 43200 // number in seconds - 2 times a day

 /*

  Number of retries if first time server request fails.

 */

@@ -190,7 +198,7 @@ SoftwareSerial MightyOhmTxOut(5, 6);

 */

 char mightOhmData[MIGHTYOHM_DATA_STRING_LENGTH + 1];

 unsigned long nextSerialUpdateTime = 0;

-time_t nextClockSynchTime = -1;

+unsigned long nextClockSynchTime = 0;

 /*

  Create global variables to store the verbose mode state (ON or OFF)

  and the IP address mode state (static or DHCP).

@@ -240,14 +248,11 @@ void setup()

     }

   }

   Serial.print(F("IP address: ")); Serial.println(Ethernet.localIP());

-  /*

-   Start up NTP client service.

-  */

-  Udp.begin(NTP_PORT);

   /*

     Synchronize the system clock to network time.

   */

   synchronizeSystemClock();

+  nextClockSynchTime = now() + NTP_SYNCH_INTERVAL;

   /*

    Start up the HTTP server.

   */

@@ -310,7 +315,7 @@ void loop() {

      Set the time for the next network NTP

      time synchronization to occur.

     */

-    nextClockSynchTime = now() + NET_SYNCH_INTERVAL;

+    nextClockSynchTime = now() + NTP_SYNCH_INTERVAL;

   }

   /*

@@ -323,36 +328,6 @@ void loop() {

   #endif

 }

-/*

- Synchronize the local system clock to

- network time provided by NTP time server.

-*/

-void synchronizeSystemClock()

-{

-  byte count;

-  

-  Serial.println(F("Synchronizing with network time server..."));

-

-  count = 0;

-  while (1)  // Attempt to synchronize 3 times

-  {

-    if (syncToNetworkTime() == 1) {

-      //  Synchronization successful

-      break;

-    }

-    if (count == TIME_SERVER_REQUEST_RETRIES) {

-      Serial.print(F("synch failed: "));

-      break;

-    }

-    count++;

-    delay(2000);

-  }

-  if (count > 0) {

-    Serial.print(count);Serial.println(F(" retries"));

-  }

-  return;

-}

-

 /*

   Handle HTTP GET requests from an HTTP client.

 */  

@@ -376,12 +351,12 @@ void listenForEthernetClients()

     firstLineFound = false;

     /*

-     * The beginning and end of an HTTP client request is always signaled

-     * by a blank line, that is, by two consecutive line feed and carriage 

-     * return characters "\r\n\r\n".  The following lines of code 

-     * look for this condition, as well as the url extension (following

-     * "GET").

-     */

+     The beginning and end of an HTTP client request is always signaled

+     by a blank line, that is, by two consecutive line feed and carriage 

+     return characters "\r\n\r\n".  The following lines of code 

+     look for this condition, as well as the url extension (following

+     "GET").

+    */

     while (client.connected())  {

       if (client.available()) {

@@ -417,7 +392,9 @@ void listenForEthernetClients()

             i = 1;

           }

-          if (firstLineFound && (c == '\n' || i > REQUEST_STRING_BUFFER_LENGTH - 2)) {

+          if (firstLineFound && (c == '\n' || i >

+              REQUEST_STRING_BUFFER_LENGTH - 2))

+          {

             processedCommand = true;

           }

         }

@@ -447,10 +424,13 @@ void listenForEthernetClients()

       else 

         transmitErrorPage(client);

     }

-

-    //Serial.println(mightOhmData);

+    client.println();

+    #ifdef DEBUG

+      Serial.println(mightOhmData);  //debug

+    #endif

+    

     // give the web browser time to receive the data

-    delay(10);

+    delay(20);

     // close the connection:

     client.stop();

   }

@@ -478,17 +458,17 @@ void transmitWebPage(EthernetClient client) {

   strcpy(strBuffer, mightOhmData);

   /*

-   * Send the actual HTML page the user will see in their web

-   * browser.

+   Send the actual HTML page the user will see in their web

+   browser.

   */

-  client.print(F("<!DOCTYPE HTML>" \

+  client.print(F("<!DOCTYPE HTML>"                               \

                  "<html><head><title>Radiation Monitor</title>"  \

-                 "<style>pre {font: 16px arial, sans-serif;}" \

-                 "p {font: 16px arial, sans-serif;}"

-                 "h2 {font: 24px arial, sans-serif;}</style>" \

-                 "</head><body><h2>Radiation Monitor</h2>" \

-                 "<p><a href=\"http://intravisions.com/\">" \

-                 "<i>IntraVisions.com</i></a></p>" \

+                 "<style>pre {font: 16px arial, sans-serif;}"    \

+                 "p {font: 16px arial, sans-serif;}"             \

+                 "h2 {font: 24px arial, sans-serif;}</style>"    \

+                 "</head><body><h2>Radiation Monitor</h2>"       \

+                 "<p><a href=\"http://intravisions.com/\">"      \

+                 "<i>IntraVisions.com</i></a></p>"               \

                  "<hr>"));

   /* Data Items */             

   client.print(F("<pre>UTC &#9;"));

@@ -520,8 +500,8 @@ void transmitRawData(EthernetClient client) {

   strcpy(strBuffer, mightOhmData);

   /*

-   * Format and transmit a JSON compatible data string.

-   */

+   Format and transmit a JSON compatible data string.

+  */

   client.print(F("$,UTC="));

   client.print(strtok(strBuffer, " "));

   client.print(F(" "));

@@ -545,13 +525,13 @@ void transmitRawData(EthernetClient client) {

 }

 /*

- * Send an error message web page back to the requesting

- * client when the client provides an invalid url extension.

- */

+ Send an error message web page back to the requesting

+ client when the client provides an invalid url extension.

+*/

 void transmitErrorPage(EthernetClient client) {

-  client.print(F("<!DOCTYPE HTML>" \

+  client.print(F("<!DOCTYPE HTML>"                                      \

                  "<html><head><title>Radiation Monitor</title></head>"  \

-                 "<body><h2>404 Not Found</h2>"  \

+                 "<body><h2>404 Not Found</h2>"                         \

                  "</body></html>"

                  ));

 }

@@ -584,7 +564,7 @@ void processRxByte( char RxByte )

      characters are not preserved in the temporary read buffer, so

      restore them to the MightyOhm data buffer, as well.

     */

-    sprintf( mightOhmData, "%d:%02d:%02d %d/%d/%d, %s",       \

+    sprintf( mightOhmData, "%d:%02d:%02d %d/%d/%d, %s",        \

           hour(), minute(), second(), month(), day(), year(),  \

           MIGHTYOHM_DATA_STRING_HEADER );

     /*

@@ -624,6 +604,48 @@ void processRxByte( char RxByte )

   return;

 } 

+/*

+ Synchronize the local system clock to

+ network time provided by NTP time server.

+*/

+void synchronizeSystemClock()

+{

+  byte count;

+

+  Serial.print(F("Synchronizing with NTP server: "));

+  Serial.print(timeServer);Serial.println(F("..."));

+

+  /*

+   * NOTICE!!!    NOTICE!!!   NOTICE!!!

+   * Due to a bug in the Ethernet library, it is necessary to reinitialize 

+   * the ethernet UDP library everytime after an  after an EthernetClient 

+   * class object has been instantiated.  Also, the Udp stop() function 

+   * must be called at the end of each session.

+   */

+  Udp.begin(NTP_PORT);  // see above comment

+

+  count = 1;

+  while (true)  // Attempt to synchronize 3 times

+  {

+    if (syncToNetworkTime() == 1) {

+      //  Synchronization successful

+      break;

+    }

+    if (count == TIME_SERVER_REQUEST_RETRIES) {

+      Serial.print(F("synch failed: "));

+      break;

+    }

+    count++;

+    delay(2000);

+  }

+  if (count > 1) {

+    Serial.print(count);Serial.println(F(" retries"));

+  }

+  

+  Udp.stop(); // see above comment

+  return;

+}

+

 /* 

   Send a UDP request packet to an NTP time server and listen for a reply.

   When the reply arrives, parse the received UPD packet and compute unix

@@ -632,9 +654,10 @@ void processRxByte( char RxByte )

 int syncToNetworkTime()

 {

   /*

-   Send a request to the NTP time server.

+   Send a request to the NTP time server.  Define a buffer to hold outgoing

+   and incoming packets.

   */

-  byte packetBuffer[ NTP_PACKET_SIZE]; //buffer to hold outgoing and incoming packets 

+  byte packetBuffer[ NTP_PACKET_SIZE]; // buffer to hold packets

   /*

    Send an NTP packet to the time server and allow for network lag

    before checking if a reply is available.

@@ -652,8 +675,8 @@ int syncToNetworkTime()

     */

     Udp.read( packetBuffer, NTP_PACKET_SIZE );

     /*

-     The timestamp starts at byte 40 of the received packet and is four bytes,

-     or two words, long. First, esxtract the two words.

+     The timestamp starts at byte 40 of the received packet and is four

+     bytes, or two words, long. First, esxtract the two words.

     */

     unsigned long highWord = word(packetBuffer[40], packetBuffer[41]);

     unsigned long lowWord = word(packetBuffer[42], packetBuffer[43]);

@@ -681,7 +704,7 @@ int syncToNetworkTime()

 }

 /*

-  Send an NTP request to the NTP time server.

+ Send an NTP request to the NTP time server.

 */

 void sendNTPpacket( char * serverAddress, byte * packetBuffer )

 {

@@ -706,8 +729,8 @@ void sendNTPpacket( char * serverAddress, byte * packetBuffer )

   /*

    All NTP fields have been given values, so now

    send a packet requesting a timestamp.

-  */ 		

-  Udp.beginPacket( serverAddress, 123 ); //NTP requests are to port 123

+  */

+  Udp.beginPacket( serverAddress, 123 ); // NTP requests are to port 123

   Udp.write( packetBuffer, NTP_PACKET_SIZE );

   Udp.endPacket();

   return;

@@ -724,31 +747,27 @@ void commandMode()

 {

   char sCmdBuf[2];

-  getCurrentIP();  //used for display of settings

+  getCurrentIP();  // used for display of settings

+

+  Serial.println();

+  displayMenu(); // display the menu

   while(true)

   {

-    /*

-     Print the menu.

-    */

-    Serial.print( F("\n"                         \

-                  "1 - view settings\r\n"        \

-                  "2 - set IP address\r\n"       \

-                  "3 - set NTP server\r\n"       \

-                  "4 - toggle verbose\r\n"       \

-                  "5 - exit without saving\r\n"  \

-                  "6 - save & restart\r\n"       \

-                  ">"));

     /*

      Get the command from the user.

     */

+    Serial.print(F(">"));

     getSerialLine(sCmdBuf, 2);

-    Serial.print(F("\n\n\r"));

+    Serial.print(F("\n\r"));

     /* 

      Execute the command.

     */

     switch (sCmdBuf[0])

     {

+      case '0':

+        displayMenu();

+        break;

       case '1':

         displaySettings();

         break;

@@ -782,6 +801,25 @@ void commandMode()

   return;

 }

+/*

+ Displays the menu.

+*/

+void displayMenu()

+{

+  /*

+   Print the menu.

+  */

+  Serial.print( F("Available commands (type a number):\r\n" \

+                  "  0 - display this menu\r\n"    \

+                  "  1 - view settings\r\n"        \

+                  "  2 - set IP address\r\n"       \

+                  "  3 - set NTP server\r\n"       \

+                  "  4 - toggle verbose\r\n"       \

+                  "  5 - exit without saving\r\n"  \

+                  "  6 - save & restart\r\n"       \

+              ));  

+}

+

 /*

  Displays the current system settings.  Displays

  RadMon software version, local IP address, NTP server

@@ -853,9 +891,9 @@ void setNTPServer()

 {

   char sBuf[32];

-  Serial.print(F("enter IP (<CR> for default): "));

+  Serial.print(F("enter NTP server (<CR> for default): "));

   getSerialLine(sBuf, 32);

-  

+

   if (strlen(sBuf) == 0)

   {

     strcpy(timeServer, DEFAULT_NTP_SERVER_ADDR);

@@ -72,7 +72,7 @@

    * v17 released 29 Oct 2019 by J L Owrey

        - modified NTP server address user setting to allow fully

          qualified domain names as well as IP addresses.  Default

-         NTP address set to "time.nist.gov" per NIST request to use

+         NTP address set to "pool.ntp.org" per ntp.org request to use

          (in order to facilitate load balancing) the fully qualified

          domain name instead of individual server IP addresses.

 */

@@ -68,7 +68,13 @@

        - simplified serial data output

    * v16 released 16 Sep 2017 by J L Owrey

        - added capability of rebooting via network http request,

-         i.e.,    "http://{device_IP_address}/reset"

+         i.e., "http://{device IP address}/reset"

+   * v17 released 29 Oct 2019 by J L Owrey

+       - modified NTP server address user setting to allow fully

+         qualified domain names as well as IP addresses.  Default

+         NTP address set to "time.nist.gov" per NIST request to use

+         (in order to facilitate load balancing) the fully qualified

+         domain name instead of individual server IP addresses.

 */

 /***  PREPROCESSOR DEFINES  ***/

@@ -79,8 +85,8 @@

  Define the header and version number displayed at startup

  and also by the 'view settings' command.

 */

-#define STARTUP_HEADER "\n\rRadmon v1.6 (c) 2018\n"

-#define RADMON_VERSION "v1.6"

+#define STARTUP_HEADER "\n\rRadmon v1.7 (c) 2019\n"

+#define RADMON_VERSION "v1.7"

 /*

  The following define sets the MAC address of the device.  This

  address is a permanent attribute of the device's Ethernet interface,

@@ -89,7 +95,7 @@

  specific instance of the Ethernet shield.  This MAC address should

  be shown on a label affixed to the device housing.

 */

-#define ETHERNET_MAC_ADDRESS 0x90, 0xA2, 0xDA, 0x0D, 0x84, 0xF6

+#define ETHERNET_MAC_ADDRESS 0xNN, 0xNN, 0xNN, 0xNN, 0xNN, 0xNN

 /*

  The following defines an APIPA default address in the event that

  DHCP mode is ON and a DHCP address cannot be obtained.

@@ -114,14 +120,18 @@

  time server:

               time-c-b.nist.gov

 */

-#define DEFAULT_NTP_SERVER_IP_ADDR "132.163.96.3"

+#define DEFAULT_NTP_SERVER_ADDR "pool.ntp.org"

 #define NTP_PORT 8888

 #define NTP_PACKET_SIZE 48 // NTP time stamp is in the first 48 bytes of the message

 /*

  The following defines how often the system clock gets synchronized

  to network time.

 */

-#define NET_SYNCH_INTERVAL 43200 //number in seconds

+#define NET_SYNCH_INTERVAL 21600 //number in seconds - 4 times a day

+/*

+ Number of retries if first time server request fails.

+*/

+#define TIME_SERVER_REQUEST_RETRIES 3

 /*

  The following defines the size of the buffer space required for the

  serial data string from the Mighty Ohm Geiger counter.  The serial

@@ -179,8 +189,8 @@ SoftwareSerial MightyOhmTxOut(5, 6);

  time, and next synchronization time.

 */

 char mightOhmData[MIGHTYOHM_DATA_STRING_LENGTH + 1];

-unsigned long lastSerialUpdateTime;

-time_t nextClockSynchTime;

+unsigned long nextSerialUpdateTime = 0;

+time_t nextClockSynchTime = -1;

 /*

  Create global variables to store the verbose mode state (ON or OFF)

  and the IP address mode state (static or DHCP).

@@ -189,10 +199,10 @@ boolean bVerbose;

 boolean bUseStaticIP;

 /*

  Create and initialize global arrays to hold the current IP address

- and the NTP server IP address.

+ and the NTP server address.

 */

 byte ipAddr[4];

-byte ntpIpAddr[4];

+char timeServer[32];

 /*** SYSTEM STARTUP  ***/

@@ -215,11 +225,13 @@ void setup()

    Start up the Ethernet interface using either a static or

    DHCP supplied address (depending on stored system configuration).

   */

-  if(bUseStaticIP)

+  if (bUseStaticIP)

   {

     Ethernet.begin(mac, ipAddr);

-  } else {

-    if ( Ethernet.begin(mac) == 0 )

+  }

+  else

+  {

+    if (Ethernet.begin(mac) == 0)

     {

       /* DHCP not responding so use APIPA address */

       parseIpAddress(ipAddr, DEFAULT_APIPA_IP_ADDRESS);

@@ -245,12 +257,6 @@ void setup()

     Open serial communications to the MightyOhm device.

   */  

   MightyOhmTxOut.begin(9600);

-   /*

-    Initialize initial time for sending out the hearbeat string. Normally

-    the system clock will be at approx 3200 msec at this point. So allow

-    some additional time for data to accumulate in MightyOhm data buffer.

-  */

-  lastSerialUpdateTime = -1;

   /*

    Initialize MightyOhm data string to empty.

   */

@@ -261,17 +267,13 @@ void setup()

 /*** MAIN LOOP ***/

 void loop() {

-  long currentTime;

-

-  currentTime = millis();

-  

   /*

    Check for user keyboard 'c' pressed.  This character switches

    to command mode.

   */   

-  if ( Serial.available() ) {

+  if (Serial.available()) {

     // get incoming byte

-    if(Serial.read() == 'c') {

+    if (Serial.read() == 'c') {

       commandMode();

     }

   }

@@ -280,8 +282,8 @@ void loop() {

     Poll serial input buffer from MightyOhm for new data and 

     process received bytes to form a complete data string.

   */

-  while ( MightyOhmTxOut.available() ) {

-    processRxByte( MightyOhmTxOut.read() );

+  while (MightyOhmTxOut.available()) {

+    processRxByte(MightyOhmTxOut.read());

   }

   /*

@@ -289,9 +291,12 @@ void loop() {

     serial port at regular intervals.

   */

   if (bVerbose) {

-    if (abs(millis() - lastSerialUpdateTime) > SERIAL_UPDATE_INTERVAL) {

-      lastSerialUpdateTime = millis();

-      Serial.println( mightOhmData );

+    if (millis() > nextSerialUpdateTime) {

+      Serial.println(mightOhmData);

+      /* 

+       Set the time for the next serial update to occur.

+      */

+      nextSerialUpdateTime = millis() + SERIAL_UPDATE_INTERVAL;

     }

   }

@@ -299,8 +304,13 @@ void loop() {

    Periodically synchronize local system clock to time

    provided by NTP time server.

   */

-  if ( now() > nextClockSynchTime ) {

+  if (now() > nextClockSynchTime) {

     synchronizeSystemClock();

+    /* 

+     Set the time for the next network NTP

+     time synchronization to occur.

+    */

+    nextClockSynchTime = now() + NET_SYNCH_INTERVAL;

   }

   /*

@@ -322,24 +332,24 @@ void synchronizeSystemClock()

   byte count;

   Serial.println(F("Synchronizing with network time server..."));

-    

-  for(count = 0; count < 3; count++)  // Attempt to synchronize 3 times

+

+  count = 0;

+  while (1)  // Attempt to synchronize 3 times

   {

-    if(syncToNetworkTime() == 1)

-    {

+    if (syncToNetworkTime() == 1) {

       //  Synchronization successful

       break;

     }

-    delay(1000);

-  } /* end for */ 

-  if(count == 3) {

-    Serial.println(F("synch failed"));

+    if (count == TIME_SERVER_REQUEST_RETRIES) {

+      Serial.print(F("synch failed: "));

+      break;

+    }

+    count++;

+    delay(2000);

+  }

+  if (count > 0) {

+    Serial.print(count);Serial.println(F(" retries"));

   }

-  /* 

-   Set the time for the next network NTP

-   time synchronization to occur.

-  */

-  nextClockSynchTime = now() + NET_SYNCH_INTERVAL;

   return;

 }

@@ -477,8 +487,8 @@ void transmitWebPage(EthernetClient client) {

                  "p {font: 16px arial, sans-serif;}"

                  "h2 {font: 24px arial, sans-serif;}</style>" \

                  "</head><body><h2>Radiation Monitor</h2>" \

-                 "<p><a href=\"http://intravisions.com/radmon/\">" \

-                 "<i>intravisions.com/radmon</i></a></p>" \

+                 "<p><a href=\"http://intravisions.com/\">" \

+                 "<i>IntraVisions.com</i></a></p>" \

                  "<hr>"));

   /* Data Items */             

   client.print(F("<pre>UTC &#9;"));

@@ -541,10 +551,7 @@ void transmitRawData(EthernetClient client) {

 void transmitErrorPage(EthernetClient client) {

   client.print(F("<!DOCTYPE HTML>" \

                  "<html><head><title>Radiation Monitor</title></head>"  \

-                 "<body><h2>Invalid Url</h2>"  \

-                 "<p>You have requested a service at an unknown " \

-                 "url.</p><p>If you think you made this request in error, " \

-                 "please disconnect and try your request again.</p>" \

+                 "<body><h2>404 Not Found</h2>"  \

                  "</body></html>"

                  ));

 }

@@ -632,12 +639,12 @@ int syncToNetworkTime()

    Send an NTP packet to the time server and allow for network lag

    before checking if a reply is available.

   */

-  sendNTPpacket(packetBuffer);

-  delay(2000);  // allow 2000 milli-seconds for network lag

-

+  sendNTPpacket(timeServer, packetBuffer);

   /*

    Wait for response from NTP time server.

   */

+  delay(1000);  // allow 1000 milli-seconds for network lag

+

   if ( Udp.parsePacket() )

   {  

     /*

@@ -676,7 +683,7 @@ int syncToNetworkTime()

 /*

   Send an NTP request to the NTP time server.

 */

-void sendNTPpacket( byte* packetBuffer )

+void sendNTPpacket( char * serverAddress, byte * packetBuffer )

 {

   /*

    Set all bytes in the buffer to 0.

@@ -700,7 +707,7 @@ void sendNTPpacket( byte* packetBuffer )

    All NTP fields have been given values, so now

    send a packet requesting a timestamp.

   */ 		

-  Udp.beginPacket( ntpIpAddr, 123 ); //NTP requests are to port 123

+  Udp.beginPacket( serverAddress, 123 ); //NTP requests are to port 123

   Udp.write( packetBuffer, NTP_PACKET_SIZE );

   Udp.endPacket();

   return;

@@ -749,7 +756,7 @@ void commandMode()

         setIP();

         break;

       case '3':

-        setNTPIP();

+        setNTPServer();

         break;

       case '4':

         toggleVerbose();

@@ -765,6 +772,7 @@ void commandMode()

          server or to initialize the Ethernet interface

          with a static IP address.

         */

+        delay(100);

         software_Reset();

         return;

       default:

@@ -801,9 +809,8 @@ void displaySettings()

   Serial.println(sBuf);

   // Display NTP server IP address

-  sprintf(sBuf, "%d.%d.%d.%d", ntpIpAddr[0], ntpIpAddr[1], ntpIpAddr[2], ntpIpAddr[3]);

   Serial.print(F("NTP server: ")); 

-  Serial.println(sBuf);

+  Serial.println(timeServer);

   // Display verbose mode setting

   printVerboseMode();

@@ -842,21 +849,20 @@ void setIP()

  carriage return as the first character, then use the

  default IP address for the NTP server.

 */

-void setNTPIP()

+void setNTPServer()

 {

-  char sBuf[16];

+  char sBuf[32];

   Serial.print(F("enter IP (<CR> for default): "));

-  getSerialLine(sBuf, 16);

+  getSerialLine(sBuf, 32);

   if (strlen(sBuf) == 0)

   {

-    strcpy(sBuf, DEFAULT_NTP_SERVER_IP_ADDR);

-    parseIpAddress(ntpIpAddr, sBuf);

+    strcpy(timeServer, DEFAULT_NTP_SERVER_ADDR);

   }

   else

   {

-    parseIpAddress(ntpIpAddr, sBuf);

+    strcpy(timeServer, sBuf);

   }

   Serial.println();

   return;

@@ -991,13 +997,20 @@ char* getSerialLine(char* sBuffer, int bufferLength)

 void writeSettingsToEEPROM()

 {

   byte ix;

+  char c;

   for (ix = 0; ix < 4; ix++)

   {

     EEPROM.write(ix, ipAddr[ix]);

-    EEPROM.write(ix + 4, ntpIpAddr[ix]);

   }

-  EEPROM.write(8, bVerbose);

-  EEPROM.write(9, bUseStaticIP);

+  EEPROM.write(4, bVerbose);

+  EEPROM.write(5, bUseStaticIP);

+  ix = 0;

+  while(1) {

+    c = timeServer[ix];

+    EEPROM.write(6 + ix, c);

+    if (c == 0 || ix > 31) break;

+    ix++;

+  }

   return;

 }

@@ -1010,13 +1023,20 @@ void writeSettingsToEEPROM()

 void readSettingsFromEEPROM()

 {

   byte ix;

+  char c;

   for (ix = 0; ix < 4; ix++)

   {

     ipAddr[ix] = EEPROM.read(ix);

-    ntpIpAddr[ix] = EEPROM.read(ix + 4);

   }

-  bVerbose = EEPROM.read(8);

-  bUseStaticIP = EEPROM.read(9);

+  bVerbose = EEPROM.read(4);

+  bUseStaticIP = EEPROM.read(5);

+  ix = 0;

+  while(1) {

+    c = EEPROM.read(6 + ix);

+    timeServer[ix] = c;

+    if (c == 0 || ix > 31) break;

+    ix++;

+  }

   return;

 }

@@ -1046,4 +1066,3 @@ void software_Reset()

   asm volatile ("  jmp 0");

   return; 

 }  

-

@@ -68,7 +68,7 @@

        - simplified serial data output

    * v16 released 16 Sep 2017 by J L Owrey

        - added capability of rebooting via network http request,

-         i.e., "http://{device IP address}/reset"

+         i.e.,    "http://{device_IP_address}/reset"

 */

 /***  PREPROCESSOR DEFINES  ***/

@@ -121,7 +121,7 @@

  The following defines how often the system clock gets synchronized

  to network time.

 */

-#define NET_SYNCH_INTERVAL 21600 //number in seconds

+#define NET_SYNCH_INTERVAL 43200 //number in seconds

 /*

  The following defines the size of the buffer space required for the

  serial data string from the Mighty Ohm Geiger counter.  The serial

@@ -191,8 +191,6 @@ boolean bUseStaticIP;

  Create and initialize global arrays to hold the current IP address

  and the NTP server IP address.

 */

-//byte ipAddr[4] = {};

-//byte ntpIpAddr[4] = {};

 byte ipAddr[4];

 byte ntpIpAddr[4];

@@ -440,7 +438,7 @@ void listenForEthernetClients()

         transmitErrorPage(client);

     }

-    Serial.println(mightOhmData);

+    //Serial.println(mightOhmData);

     // give the web browser time to receive the data

     delay(10);

     // close the connection:

@@ -480,7 +478,7 @@ void transmitWebPage(EthernetClient client) {

                  "h2 {font: 24px arial, sans-serif;}</style>" \

                  "</head><body><h2>Radiation Monitor</h2>" \

                  "<p><a href=\"http://intravisions.com/radmon/\">" \

-                 "<i>IntraVisions.com/radmon</i></a></p>" \

+                 "<i>intravisions.com/radmon</i></a></p>" \

                  "<hr>"));

   /* Data Items */             

   client.print(F("<pre>UTC &#9;"));

@@ -453,10 +453,10 @@ void listenForEthernetClients()

  requesting client,

 */

 void transmitHttpHeader(EthernetClient client) {

-  client.print(F("HTTP/1.1 200 OK\r\n"        \

+  client.print(F("HTTP/1.1 200 OK\r\n"          \

                  "Content-Type: text/html\r\n"  \

                  "Connnection: close\r\n"       \

-                 "Refresh: 10\r\n"              \

+                 "Refresh: 5\r\n"               \

                  "\r\n"                         \

                  ));

 }

@@ -12,7 +12,7 @@

  network interface IP address, NTP server address, or configure a

  verbose output mode.

- Copyright 2017 Jeff Owrey