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
> 

Reply via email to