How to get a JSON payload from a php page using a MKR1010?

[badakzz]

Hello guys,

So I've been struggling on this for a while. My goal is to create an alarm using the MKR 1010, by in the first time getting the local current time from SSID, and then comparing it to a JSON payload.
However, I cannot seem to get the payload right. I've tried a lot of things but none of them have worked.
Please bear with me, I'm quite new to coding !
Right now I do have a JSON payload which URL is http://hiddendomainname.com/post/json/json_playload.php.

It is structured as such :

Here's my code so far :

#include <SPI.h>
#include <WiFiNINA.h>
#include <WiFiUdp.h>
#include <ArduinoJson.h>
#include <ArduinoHttpClient.h>


int status = WL_IDLE_STATUS;
///////please enter your sensitive data in the Secret tab/arduino_secrets.h
char ssid[] = SECRET_SSID;        // your network SSID (name)
char pass[] = SECRET_PASS;    // your network password (use for WPA, or use as key for WEP)
int keyIndex = 0;            // your network key Index number (needed only for WEP)
int setTime = 0;
int motorPin = 3;

char serverAddress[] = "hiddendomainname.com";  
int port = 80;
WiFiClient wifi;
HttpClient client= HttpClient(wifi, serverAddress, port);

unsigned int localPort = 2390;      // local port to listen for UDP packets

IPAddress timeServer(129, 6, 15, 28); // time.nist.gov NTP server

const int NTP_PACKET_SIZE = 48; // NTP time stamp is in the first 48 bytes of the message

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

// A UDP instance to let us send and receive packets over UDP

WiFiUDP Udp;
WiFiServer server(80);

void setup() {

  // Open serial communications and wait for port to open:

  Serial.begin(9600);
  pinMode(motorPin, OUTPUT);

  while (!Serial) {

    ; // wait for serial port to connect. Needed for native USB port only

  }

  // check for the WiFi module:

  if (WiFi.status() == WL_NO_MODULE) {

    Serial.println("Communication with WiFi module failed!");

    // don't continue

    while (true);

  }

  String fv = WiFi.firmwareVersion();

  if (fv < WIFI_FIRMWARE_LATEST_VERSION) {

    Serial.println("Please upgrade the firmware");

  }

  // attempt to connect to Wifi network:

  while (status != WL_CONNECTED) {

    Serial.print("Attempting to connect to SSID: ");

    Serial.println(ssid);

    // Connect to WPA/WPA2 network. Change this line if using open or WEP network:

    status = WiFi.begin(ssid, pass);

    // wait 10 seconds for connection:

    delay(10000);

  }

  Serial.println("Connected to wifi");

  printWifiStatus();

  Serial.println("\nStarting connection to server...");

  Udp.begin(localPort);
}

void loop() {

  sendNTPpacket(timeServer); // send an NTP packet to a time server

  // wait to see if a reply is available

  delay(1000);

  if (Udp.parsePacket()) {

    Serial.println("packet received");

    // We've received a packet, read the data from it

    Udp.read(packetBuffer, NTP_PACKET_SIZE); // read the packet into the buffer

    //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]);

    // combine the four bytes (two words) into a long integer

    // this is NTP time (seconds since Jan 1 1900):

    unsigned long secsSince1900 = highWord << 16 | lowWord;

    Serial.print("Seconds since Jan 1 1900 = ");

    Serial.println(secsSince1900);

    // now convert NTP time into everyday time:

    Serial.print("Unix time = ");

    // Unix time starts on Jan 1 1970. In seconds, that's 2208988800:

    const unsigned long seventyYears = 2208988800UL;

    // subtract seventy years:

    unsigned long epoch = secsSince1900 - seventyYears;

    // print Unix time:

    Serial.println(epoch);

    // print the hour, minute and second:

    Serial.print("The UTC time is ");       // UTC is the time at Greenwich Meridian (GMT)

    Serial.print((epoch  % 86400L) / 3600); // print the hour (86400 equals secs per day)

    Serial.print(':');

    if (((epoch % 3600) / 60) < 10) {

      // In the first 10 minutes of each hour, we'll want a leading '0'

      Serial.print('0');

    }

    Serial.println((epoch  % 3600) / 60); // print the minute (3600 equals secs per minute)
    
    WiFiClient client = server.available();   // listen for incoming clients
  
    setTime = collect();

    if (epoch == setTime)
    {
        digitalWrite(motorPin, HIGH); // activate the motor
    }
    if (collect() == 0)
    {
        digitalWrite(motorPin, LOW);  // deactivate the motor
    }
    
  }

  // wait 30 seconds before asking for the time again

  delay(30000);

}

int collect() {
  Serial.println("making GET request");
  String request = "/post/json/json_payload.php"; //url of what we need on our server
  client.beginRequest();
  client.get(request);
  client.endRequest();
  //  read the status code and body of the response
  int statusCode = client.responseStatusCode(); 
  String response = client.responseBody();

  Serial.println(statusCode);
    //Serial.println(response);
   //Serial.println(response);
  Serial.flush();

  DynamicJsonDocument doc(450);
  String json = response;
          
  // Deserialize the JSON document
  DeserializationError error = deserializeJson(doc, json);

  // Test if parsing succeeds.
  if (error) {
    Serial.print(F("deserializeJson() failed: "));
    Serial.println(error.c_str());
    return (0);
  }

  int jsonId = doc["id"];
  int jsonTime= doc["time"];


  // Print values.
  Serial.println(jsonId);
  Serial.println(jsonTime);

  return (jsonTime);
}

// send an NTP request to the time server at the given address
unsigned long sendNTPpacket(IPAddress& address) {

  //Serial.println("1");

  // set all bytes in the buffer to 0

  memset(packetBuffer, 0, NTP_PACKET_SIZE);

  // Initialize values needed to form NTP request

  // (see URL above for details on the packets)

  //Serial.println("2");

  packetBuffer[0] = 0b11100011;   // LI, Version, Mode

  packetBuffer[1] = 0;     // Stratum, or type of clock

  packetBuffer[2] = 6;     // Polling Interval

  packetBuffer[3] = 0xEC;  // Peer Clock Precision

  // 8 bytes of zero for Root Delay & Root Dispersion

  packetBuffer[12]  = 49;

  packetBuffer[13]  = 0x4E;

  packetBuffer[14]  = 49;

  packetBuffer[15]  = 52;

  //Serial.println("3");

  // all NTP fields have been given values, now

  // you can send a packet requesting a timestamp:

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

  //Serial.println("4");

  Udp.write(packetBuffer, NTP_PACKET_SIZE);

  //Serial.println("5");

  Udp.endPacket();

  //Serial.println("6");
}

void printWifiStatus() {

  // print the SSID of the network you're attached to:

  Serial.print("SSID: ");

  Serial.println(WiFi.SSID());

  // print your board's IP address:

  IPAddress ip = WiFi.localIP();

  Serial.print("IP Address: ");

  Serial.println(ip);

  // print the received signal strength:

  long rssi = WiFi.RSSI();

  Serial.print("signal strength (RSSI):");

  Serial.print(rssi);

  Serial.println(" dBm");
}

And the monitor writes :

Please upgrade the firmware
Attempting to connect to SSID: FreeBox
Connected to wifi
SSID: FreeBox
IP Address: 192.168.0.19
signal strength (RSSI):-64 dBm

Starting connection to server...
packet received
Seconds since Jan 1 1900 = 3843648666
Unix time = 1634659866
The UTC time is 16:11
making GET request
-3

Also the loop runs very slow (about 70 seconds instead of 30) and I don't know why, and that could be a problem.

Thanks in advance, my brain is really frying on this one...

[johnwasser]

Maybe there is confusion between the global variable 'client' which is connected to the external server and the local variable named 'client' that is accepting incoming connections.

@badakzz

WiFiClient wifi;
HttpClient client= HttpClient(wifi, serverAddress, port);

    WiFiClient client = server.available();   // listen for incoming clients

[badakzz]

Do you mean that I should change their name or that there is a flaw in my logic ?

I can't try it out at the moment as I'm not home

[badakzz]

I changed my code to parse the php page and not just the JSON because I could'nt understand what was going on.