We can assume that the querystring was initialized when we reach gotweb_render_header. ok tracey@

This adds a new helper function `gotweb_link' that outputs an HTML link with the parameter correctly escaped. ok stsp@

reported by the W3C validator.

due to an error in the order of the arguments to fcgi_printf that I've introduced last time I touched gotweb_render_header the title is now "/" (or SCRIPT_NAME) and one assets (the safari pinned tab svg) is a funny URL :)

keeping GOTWEBD_REPO_CACHESIZE * sizeof(struct cache_repo) inside the struct server makes it too large for imsg on some platforms (linux at least.) Instead, store a pointer and allocate the array when it's received on the child processes. ok stsp@

ok stsp@

ok stsp@

ok stsp@

if strdup fails we fail to free the querystring Found by clang' scan-build.

dead store found by clang' scan-build.

In 95a4a5a1e8d0952ee0d2523dc31df7170d80e0b3 I forgot to fix the links for the favicons and the web manifest.

problem introduced in commit 625e5896fc9ecf87ccfc92ad2a65cd3be58f73c0

got_get_repo_tags needs some tweaking to how tag_commit is allocated too. ok stsp@

all very unlikely. part of a bigger diff that's ok stsp@

drop: - a tautological s[strlen(s)] == '\0' - a few duplicates checks - a lone if (error) when it's always NULL - an extra NULL check part of a bigger diff that's ok stsp@

- don't special case ".got": if it's a work-tree gotwebd will just fail soon enough. Suggested by stsp@ - avoid an strdup. - drop `opened' to make it easier to follow what's going on. ok stsp@

ok op@

ok tracey@

This makes gotwebd use SCRIPT_NAME instead of the DOCUMENT_ROOT fastcgi param to generate the links to the assets and changes some links so every page is consistently linked with a relative URL. It allows to drop the `root "/"' in the location for fastcgi and, as a bonus, also makes possible to run gotwebd on non "/" paths. ok stsp@

ok tracey@

ok tracey@

reported by and ok tracey@