Commits


simplify loop todo was initially there for an optimization: don't loop to MAX_USERS when you know the upper limit is todo.


simplify unveil/pledge calls


rename cgi_setpoll_on_* to cgi_poll_on_*


initialize error string to avoid returning garbage


ignore also SIGHUP SIGHUP is sent when the tty is detached and by default kills the process. When we run in the background we don't care anymore about the tty, so it should be safe for us to ignore SIGHUP. (frankly, I expected daemon(3) to do stuff like this for us).


fix CGI with new IRI parser With new IRI, parser the old assumption of path starting with ./ is no longer valid.


implement a valid RFC3986 (URI) parser Up until now I used a "poor man" approach: the uri parser is barely a parser, it tries to extract the path from the request, with some minor checking, and that's all. This obviously is not RFC3986-compliant. The new RFC3986 (URI) parser should be fully compliant. It may accept some invalid URI, but shouldn't reject or mis-parse valid URI. (in particular, the rule for the path is way more relaxed in this parser than it is in the RFC text). A difference with RFC3986 is that we don't even try to parse the (optional) userinfo part of a URI: following the Gemini spec we treat it as an error. A further caveats is that %2F in the path part of the URI is indistinguishable from a literal '/': this is NOT conforming, but due to the scope and use of gmid, I don't see how treat a %2F sequence in the path (reject the URI?).


moving declarations to header file


always check absolutify_path return


make -d handle correctly non-absolute paths before the -d option only accepted absolute paths, and this wasn't documented. Even more, with the default value of "docs" it won't work. Now it transforms all relative paths to absolute paths before going on.


reorder & fmt


logging reworked and daemonize by default The -l option was removed: now it logs on syslog if -f (foreground) is not passed.


clients certs support for CGI internally, gmid doesn’t care if the client issued a certificate, but now we pass that information to the CGI script in some new environment variables.


fix bug introduced in last version regarding full URLs gemini://example.com was incorrectly parsed.


correctly handle also non absolute URLs