Write STDERR from the CGI to the web server error log as intended.

OK florian@

If the very first fcgi STDOUT record has length 0 the cgi script
didn't send anything back. This is an internal server error.
OK reyk@

Change grammar to remove a shift/reduce conflict that was introduced
with the ssl options.
"listen on $ip port 443 ssl" turns into "listen on $ip ssl port 443".

ok florian@

Prevent a possible use after free by mimicing the s3_srvr.c fixes contributed by
Adam Langley close to three years ago, which were commited in
https://git.openssl.org/gitweb/?p=openssl.git;a=commitdiff;h=e7928282d0148af5f28fa3437a625a2006af0214

ok jsing@

Support NOTE_EOF for kqueue EVFILT_READ filters on NFS files.

committing for jsg@,  ok reyk@ tedu@ guenther@

Always zero-out the fcgi record header for STDIN data.

OK florian@

Use memset(buf instead of memset(&buf.

Pointed out by deraadt@

Limit the body size in client requests (eg. POST data) to 1M by default;
add a configuration option to change the limit.

ok florian@

Document the SSL configuration for httpd (partly based on relayd.conf(5)).

sysctl machdep.ztsscale has been unused for 7 years, so stop handling it.
s/TAB/SPACE for the wsconsctl.conf comment like we do with sysctl.conf
in MI.

"get this in fast" deraadt@
ok ratchov@ who will test it in the next few hours

Provide configuration options that allow the SSL certificate, key and
ciphers to be specified for each server.

ok deraadt@ reyk@

Also clean up the public key when it is no longer needed.

ok deraadt@ reyk@

Configure the default SSL ciphers as HIGH:!aNULL.

ok deraadt@ reyk@

Allow B64_EOF to follow a base64 padding character. This restores previous
behaviour that allows a PEM block to be fed through the base64 decoder.

Reported by Dmitry Eremin-Solenikov on tech@

ok deraadt@ tedu@

Correct some dma cleanup error paths.

While the index variables were correct the arrays of
dma handles they indexed were swapped for rx and tx.

As there are a mismatched number of rx and tx descriptors
we'd walk off the end of the rx handle array by 30 items.

ok deraadt@

fix an off by one
ok deraadt@

http POST support
with & OK reyk@

Content-Length and Content-Type are transmitted as CONTENT_LENGTH and
CONTENT_TYPE environment variables to cgi scripts, without the HTTP_
prefix.
OK reyk@

spacing

avoid displaying a NULL pointer
ok deraadt@ reyk@

The watermark exposed a bug in server_write that broke keep-alive
support.  Instead of calling server_close from server_write, we have
to proceed to the next connection by calling the error handler.

OK jsg@

Bring back the last read (done) / last write (done) messages instead of just
"done" to simplify connection debugging.

Adjust the read/write watermarks according to the TCP send buffer.
This fixes sending of large files.  Previously, httpd was reading the
input file too quickly and could run out of memory when filling the
input buffer.

Found by jsg@
OK florian@

Add braces.  Style-only change.

Add an overview of the features for httpd in the description section.

"commit" deraadt@

add missing va_start/va_end calls
ok deraadt@ guenther@

Correct error checks in EVP_read_pw_string_min(): UI_add_input_string()
and UI_add_verify_string() return -1 (and maybe -2?) on failure and
>=0 on success, instead of always zero on success

problem reported by Mark Patruck (mark (at) wrapped.cx)
ok miod@

Add signify instructions plus miniroot and install56.fs for amd64/i386

ok deraadt@

Explain the options in httpd.8

ok deraadt@

Load the SSL public/private keys in the parent process, then provide them
to the privsep process via imsg. This allows the keys to be moved out of
the chroot (now /etc/ssl/server.crt, /etc/ssl/private/server.key).

ok reyk@

Add support for loading the public/private key from memory, rather than
directly from file.

Only check if the abort transfer is the interrupt one if the pipe is
opened with a callback.

If a driver opens an interrupt pipe without callback function, like
umct(4) does with one of its bulk in endpoints being reported as an
interrupt endpoint, then we can end up aborting a transfer which is
different from the interrupt one.

Issue reported by Roberto E. Vargas Caballero, ok deraadt@

Add configuration options for the most-important connection limits:
max requests (per connection) and timeout.  We don't want to add too
many button, and there are good defaults, but these ones are kind of
mandatory.

Tweak the httpd.conf manpage with "sub-lists".

Bring back the tcp/ip configuration options.  This code was already
there and is from relayd.  We can decide later which options should
be added or removed, but it shouldn't do any harm.

Add srv_conf helper variable to make the code more readable.
No functional change.

Fix an example: hostapd table entries have to be comma-separated.

From "Vigdis" via misc@
can go in deraadt@

Fix an example, nat-to requires to specify the "out" direction in pf rules.

From "Vigdis" via misc@
can go in deraadt@

Limit the number of (Keep-Alive) requests per connection to 100.
(Same default as in nginx and Apache).

Improve logging to allow per- server/location log files.  The log
files can also be owned by root now: they're opened by the parent and
send to the logger process with fd passing.  This also works with reload.

ok deraadt@

retire blink because this is serious software now; ok beck

spaces

handle wsconsctl.conf and sysctl.conf just being examples.
from ratchov

Add $OpenBSD$ tags.

add a config option to specify the chroot directory
ok reyk@

enable httpd; ok deraadt@

Temporarily move the default location of the SSL/TLS server key and
certificate from /var/www/ to /var/www/conf/.  Don't get scared - this
will be changed soon!  They're currently located in the chroot
directory but will be moved outside as soon as we adopted some of the
key privsep from relayd in ressl/httpd.

Add HTTPS = on CGI variable.

Add HTTPS server example.

Redirect to https:// if SSL/TLS is enabled.

Proxy commit for jsing@:
"Add TLS/SSL support to httpd, based on the recent ressl commits."

From jsing@
ok reyk@

manpage tweaks about logging

Implement ressl_accept_socket, which allocates a new server connection
context (if necessary) and handles the TLS/SSL handshake over the given
socket.

Return -1 on error (not 1).

A ressl server needs different configuration from a ressl client - provide
a specific server configuration function and call this from
ressl_configure.

Change grammar from "log [style]" to "log style [style]".

Provide a function that returns a server connection context.

Provide a utility function for loading a private/public keypair.

Print error message if the log files cannot be opened.

Improve ressl_{read,write} handling of non-blocking reads/writes.

Add initial support for log files in /var/www/logs/.  Logging with
syslog is still supported but disabled by default.

ok deraadt@

Free the SSL context first and let the reference counting do its thing.

Implement PATH_INFO and add DOCUMENT_ROOT.
PATH_INFO was requested by naddy@ who successfully tested it with "cvsweb".

ok naddy@

fix small layout inconsistency

ok'd by many

httpd doesn't support SSL/TLS yet, remove the remaining bits.
The secrect plan is to add it later using the ressl wrapper library.

no need for param.h

whitespace

In chacha_init(), allow for a NULL iv. Reported by znz on github.
ok guenther@ jsing@

Only allow GET and HEAD for static files or return 405.

ok florian@

Also write log messages, like 404 Not Found, on error.  This is a bit
tricky because we couldn't guarantee a sane state after
server_response_http() so fail hard afterwards and close the connection.

ok doug@

c-type functions / makros need a cast to unsigned char, not int
"feel free to commit" reyk@

Allocate http_host instead of carrying a buffer in the descriptor.

spacing

Dynamically pass HTTP request headers as protocol-specific HTTP_* CGI
meta-variables.

ok florian@

Add ral(4) at pci and cardbus to RAMDISK on macppc; ok deraadt

Add ral(4) at cardbus to GENERIC on macppc; ok deraadt

smtpd got privsep'ed and the _smtpq user is now used to manage these
directories. Ensure proper ownership in case smtpd has been started
before this change.

brought up by matthieu@
OK deraadt@

X509_NAME_get_text_by_NID() returns -1 on error so the type
the return value is stored in must be signed.  Fixes a test for error.

ok jsing@ guenther@

don't return unitialised memory on error in ehci_alloc_sqtd()

It seems mpi introduced a problem into ehci.c rev 1.162
two weeks ago.  An error check that used to return NULL
now jumps to the end of the function, but sqtd isn't
initialised at that point.

ok miod@ deraadt@

When stsp changed bwi_encap() for 30 bit addresses in rev 1.106 code
that used to always initialise the error variable is no longer run.

And at the end of bwi_encap() there is:

if (error)
m_freem(m);
return (error);

Fixing this prevents packet loss stsp was seeing.

ok stsp@ miod@ deraadt@

Add function to iterate all headers.  No functional change.

The first server example should be the "minimal default" to illustrate
that you don't have to push all kinds of buttons to run httpd.

More examples, include FastCGI for php and cgi-bin and logging.

Split fastcgi socket path and document root option and add the
SCRIPT_FILENAME CGI param with a prepended root.  This fixes php-fpm
that expects SCRIPT_FILENAME and also works with slowcgi if you
configure the root correctly.  For example, if SCRIPT_NAME and
REQUEST_URI are /php/index.php, root is /htdocs, SCRIPT_FILENAME will
be /htdocs/php/index.php.  As tested and discussed with florian@

Add missing log call for FastCGI requests.

Add another log mode "connection" for a relayd(8)-style log entry after
each connection, not every request.  The code was already there and enabled
on debug, I just turned it into an alternative log format.

Prefer getnameinfo() with NI_NUMERICHOST over inet_ntop because it is also
aware of the IPv6 scope Id.  We already have a function print_host() that
uses getnameinfo, so no need for the inet_ntop cases. Confirmed by florian@

Locations now inherit access log settings from the server.

Add log to the server flags.

input/"Looks ok" reyk@

don't leak fcgi fd

Padding of fcgi records is optional, but if we receive padding data we
should read it.

We need to read from the fcgi bufferevent until it's empty because the
event handler will not be called again if no new data arrives.
Debugged with and OK reyk@

Allow to specify a FastCGI TCP socket on localhost (eg. :9000).  Used
for debugging, you should prefer local UNIX sockets, but it helped to
find an issue that will be fixed with the next commit.

OK florian@

'fastcgi socket "path"' is the correct syntax; update the manpage.
Found by jsg@

spacing

scandir(3)-based directory auto index didn't work on NFS because the
file system is not filling in d_type properly.  Using st_mode from the
stat call fixes the problem, eg. S_ISDIR(st.st_mode) instead of
dp->d_type == DT_DIR.  Pointed out by pelikan@

remove nasty unclosed Xo in previous; ok reyk

Use the log buffer to defer the logging until the connection is closed
or the request completed.  Turn the old log message into a debug message.

ok doug@

remove the global "log updates/all" option that came from relayd.

Add common and combined access logging to httpd.

ok reyk@

Rewrite fcgi_add_param and hand over a lot more http headers etc. to
the cgi script.
OK reyk@
"blanket OK" for changes in httpd for the time beeing from deraadt@