teleserver: reduces teleloger-timestamp(ticks) from 64bit-->32bit and overflows every 2 weeks

[ankostis]

(i'm surprised nobody else has reported this)

The telelogger (v5) transmits uint64 for timestamps from CPU ticks

Freematics/firmware_v5/telelogger/teleclient.cpp

Line 161 in 98f38d9

netbuf.dispatch(buf, sprintf(buf, "TS=%lu", millis()));

but teleserver although it uses atol() to parse it back, it assigns the result in a uint32(!):

Freematics/server/teleserver/teleserver.h

Line 87 in 98f38d9

uint32_t deviceTick;

Freematics/server/teleserver/udpserver.c

Line 115 in 98f38d9

uint32_t deviceTick = 0;

Freematics/server/teleserver/udpserver.c

Lines 129 to 131 in 98f38d9

	else if (!strncmp(s, "TS=", 3)) {
	deviceTick = atol(s + 3);
	}

Indeed we have noticed that device-ticks overflow at random durations, roughly before ~20 days (31bits correspond to ~25days, 32-->49days).

Q1: is this analysis correct?
Q2: (if yes) is this on purpose?
Q3: (if no) would you accept a PR to fix it?