goatcounter monitor fails with "syntax error" with sqlite

[anarcat]

I get this when running goatcounter monitor:

   16:37:18 monitor: ERROR: near "'120 seconds'": syntax error
   16:37:18 monitor: ERROR: no hits

I figured maybe something was up with defaults, so I also tried overriding the -period:

root@marcos:~# docker run -it --rm --volume="goatcounter:/home/user/db/" zgoat/goatcounter monitor -period 120
   16:37:18 monitor: ERROR: near "'120 seconds'": syntax error
   16:37:18 monitor: ERROR: no hits

(Note that I use Docker, above, but that should be equivalent to just running goatcounter monitor -period 120.)

The errors comes back once in a while, about every two minutes...

It seems like this is a problem with this database query:

	query += ` created_at > now() - interval '%d seconds'`

I suspect that has been designed for PostgreSQL and does not work with sqlite. Apparently, that only works in MySQL (and PostgreSQL i guess?)... the sqlite-way of doing this seems to be, according to SO:

SELECT mumble FROM blah WHERE blah.heart_beat_time > datetime('now', '-600 seconds');

... and obviously, that is a SQLite-specific extension that doesn't work in the other two...whee!

[arp242]

I just created this quickly after an outage several months ago to make sure I would be notified ASAP, kinda forgot about SQLite 😅 There's already a little helper to deal with this specific syntax difference. Should be a simple fix.