their return value is no longer used, it's only confusing at this point.

instead of having a flag to discern between two different behaviours in S_SENDING, split that state into S_SENDING_FILE and S_SENDING_CGI (this will also make it easier in the future to add other sending states). While there, also get rid of `goodbye' and make start_reply advance the state machine by itself.

tested on openbsd, alpine and void

now we log the full IRI requested (before was only the path) and the response line (even for CGI).

was probably added to distinguish between AF_INET and AF_INET6, to use different functions for logging. But it wasn't really used, and now we use getnameinfo anyway.