strncpy -> strlcpy quoting strncpy(3) strncpy() only NUL terminates the destination string when the length of the source string is less than the length parameter. strlcpy is more intuitive. this is another warning gcc 8 found that clang didn't.

make sure l is always initialized I can't think of cases where we reach serialize_iri and path is NULL, but let's keep the safe side and initialize l. gcc 8 found this, clang didn't.

remove last occurrences of lex

remove now unused lex.l

move lineno/colno into the token struct

drop the dependency on lex by implementing yylex by ourselves The actual implementation is based off doas' parse.y. This gave us various benefits, like cleaner code, \ to break long lines, better handling of quotes etc...

kill unused variable leftover from the handle_imsg_log -> do_log transition.

be quieter nobody really cares if must_read fails, as it normally shouldn't. It only clutters the regression tests with scary messages that looks like failure but are, in fact, expected.

allow sending fd to log on to the logger process the logger process now can receive a file descriptor to write logs to. At the moment the logic is simple, if it receives a file it logs there, otherwise it logs to syslog. This will allow to log on custom log files.

add some simple test for fastcgi and while there reorder the targets in the Makefile a bit

add simple fcgi application for test purposes

explicitly use c->fd instead of fd Yep, fd should be the file descriptor, but for lazyness when manually calling the function sometimes we supply 0 as fd and event. Instead of fixing the usage, do as other of such functions do in this circumstances: use c->fd.

mark backend as FCGI_READY when getting a fd otherwise clients will remain stuck waiting for a pending request that doesn't exist (see apply_fastcgi switch.)

indentation

free fastcgi param list