Cleaner, more elegant logging

[ubidefeo]

ATTENTION: requires consulting the Cloud team before merging
I have refactored the logging entirely and added one extra level of verbosity (4).
It is now leaner and cleaner.
I also implemented firmware version check and compare but have an issue with WiFiNINA reporting the incorrect version (1.2.0) as latest.

[aentinger]

The pull request looks good to me ;) I've one suggestion though ... the debugMessage function could be slightly rewritten in order to accomodate variable arguments - then sprintf does not need to be used so much and the code becomes much cleaner to read.

A short example:

#include <stdio.h>
#include <stdarg.h>
...
void debugMessage(uint8_t const debug_level, char const * fmt, ...) {
  char debug_buf[64] = {0};

  va_list args;

  va_start (args, fmt);
  uint16_t const length = vsnprintf (debug_bug, 64, fmt, args);
  va_end   (args);

  Serial.print(debug_buf);
}

Usage would be then like: Instead of

*msgBuffer = 0;
sprintf(msgBuffer, "WiFi.status(): %d", WiFi.status());
debugMessage(msgBuffer, 4);

do this

debugMessage(4, "WiFi.status(): %d", WiFi.status());

[ubidefeo]

hey @lxrobotics
thanks for taking a look.
That method is being rewritten entirely because its place is definitely not as an inline in that Class.
I guess this will have to be merged before I finish that, but for now it's a sufficient way to output information.
I like the variable arguments count option a lot, didn't even think about it.
I'll make sure it's in the rewriting and then I'll work out a better implementation for the Classes I'll work on including ArduinoIoTCloud :)