Commit Briefs

Omar Polo

change log_request to take the code and meta unpacked

don't know what i was smoking when I wrote log_request() like that...




Omar Polo

implement `listen on'

Listening by default on all the addresses is so bad I don't know why I haven't changed this before. Anyway. Add a `listen on $hostname port $port' syntax to the config file and deprecate the old "port" and "ipv6" global setting. Still try to honour them when no "listen on" directive is used for backward compatibily, but this will go away in the next next version hopefully. At the moment the `listen on' in server context don't filter the host, i.e. one can still reach a host from a address not specified in the corresponding `liste on', this will be added later.



Omar Polo

typo; use the `l' variable not `len'


Omar Polo

add a privsep crypto engine

Incorporate the OpenSMTPD' privsep crypto engine. The idea behind it is to never load the certificate' private keys in a networked process, instead they are loaded in a separate process (the `crypto' one) which signs payloads on the behalf of the server processes. This way, we greatly reduce the risk of leaking the certificate' private key should the server process be compromised. This currently compiles only on LibreSSL (portable fix is in the way).


Omar Polo

parse the config file only once

Don't have all the processes read gmid.conf. The parent needs to do that, and the will send the config to the children (already happening.) The other processes were reading the config anyway to figure out the user and the chroot (if enabled); make the parent pass additional flag to propagate that info. We dissociate a bit from the "usual" proc.c but it's a change worth having.




Omar Polo

parse_conf: don't die on error, return -1

this avoids having the daemon dieing on SIGHUP with a bad config file.


Omar Polo

don't have the config being a global


Omar Polo

move hosts into the config struct


Omar Polo

move fastcgi from global var to the config struct

while here also make them a list rather than a fixed-size array.


Omar Polo

remove foreground / verbose from config

set them as global vars; rename foreground -> debug