On Sun, Aug 13, 2023 at 11:13:28AM +0200, Omar Polo wrote:
> [moving to tech@, there's a diff for the manpage below]
>
hi. comments inline:
>
> diff /usr/src
> commit - a7b17fe845fceeb2940fa5924ec5843681aa2c64
> path + /usr/src
> blob - 16b086a9ee00cd6d8e796a890e9774968556f147
> file + usr.sbin/httpd/httpd.conf.5
> --- usr.sbin/httpd/httpd.conf.5
> +++ usr.sbin/httpd/httpd.conf.5
> @@ -98,7 +98,7 @@ server "default" {
> For example:
> .Bd -literal -offset indent
> ext_ip="10.0.0.1"
> -server "default" {
> +server "example.com" {
> listen on $ext_ip port 80
> }
> .Ed
> @@ -179,7 +179,8 @@ section starts with a declaration of the server
> Each
> .Ic server
> section starts with a declaration of the server
> -.Ar name :
> +.Ar name .
> +If no one matches the request the first one defined is used.
i think it'd be clearer to use "none" rather than "no one" here. i know
what you mean, but it's easy to confuse this with the idea of no person.
also you might want to stick a comma after "request".
i.e.
If none matches the request, the first one ...
jmc
> .Bl -tag -width Ds
> .It Ic server Ar name Brq ...
> Match the server name using shell globbing rules.
> @@ -779,7 +780,7 @@ server "default" {
> .Bd -literal -offset indent
> prefork 2
>
> -server "default" {
> +server "example.com" {
> listen on * port 80
> }
>
> @@ -800,7 +801,7 @@ server "default" {
> .Qq egress
> group.
> .Bd -literal -offset indent
> -server "default" {
> +server "example.com" {
> listen on egress port 80
> }
> .Ed
>