Commit Briefs

Omar Polo

enable privsep crypto on all systems

now that we have a bundled libtls we can actually do this. Retain the knob to disable it "just in case".


Omar Polo

bundle libtls

gmid (like all other daemons that want to do privsep crypto) has a very close relationship with libtls and need to stay in sync with it. OpenBSD' libtls was recently changed to use OpenSSL' EC_KEY_METHOD instead of the older ECDSA_METHOD, on the gmid side we have to do the same otherwise failures happens at runtime. In a similar manner, privsep crypto is silently broken in the current libretls (next version should fix it.) The proper solution would be to complete the signer APIs so that applications don't need to dive into the library' internals, but that's a mid-term goal, for the immediate bundling the 'little' libtls is the lesser evil. The configure script has gained a new (undocumented for the time being) flag `--with-libtls=bundled|system' to control which libtls to use. It defaults to `bundled' except for OpenBSD where it uses the `system' one. Note that OpenBSD versions before 7.3 (inclusive) ought to use --with-libtls=bundled too since they still do ECDSA_METHOD.


Omar Polo

sync DISTFILES


Omar Polo

sync have/* files


Omar Polo

sync changelog



Omar Polo

fix INSTALL handling

Set a sane default for INSTALL, allow it to be changed either as environment variable or configure argument, and propagate it correctly to the generated config.mk. Issue reported by xavi, thanks!


Omar Polo

fix comment (ge -> gemexp)


Omar Polo

getcwd(NULL) is an extension; don't rely on it

also, while here, add some error checking too


Omar Polo

remove not so useful starts_with()

replace its only usage with strncmp(). it's likely faster too.


Omar Polo

remove a long, long unused function


Omar Polo

remove useless logging


Omar Polo

don't call client_close() from fcgi/proxy bev handlers

We might end up calling client_close() from start_reply(), but that will free the fcgi/proxy bufferevent while they're still used on the stack. Instead, start_reply() only sets REQUEST_DONE and exits, returning the error eventually, so callers know when to stop.


Omar Polo

update changelog