On 2020/11/27 13:41, Jason McIntyre wrote:
> > +++ wg.4    27 Nov 2020 12:28:32 -0000
> > @@ -64,6 +64,9 @@ interface may be configured to recognise
> >  .It Key
> >  Each peer uses its private key and corresponding public key to
> >  identify itself to others.
> > +The public key may be displayed by running
> > +.Xr ifconfig 8
> > +as root after configuring the private key.
> >  A peer configures a
> >  .Nm wg
> >  interface with its own private key and with the public keys of its peers.
> 
> i'm not sure about this text. wouldn;t the "Keys" section make more
> sense? the "Keys" section itself says:
> 
>        When an interface has a private key set with wgkey, the
>        corresponding public key is shown in the status output of
>        the interface, like so:
> 
>              wgpubkey NW5l2q2MArV5ZXpVXSZwBOyqhohOf8ImDgUB+jPtJps=
> 
> maybe we should just elaborate here?

Ah yes, that does seem a better place, maybe this helps, though
perhaps the addition to the example script is enough to give the
hint anyway.

Index: man4/wg.4
===================================================================
RCS file: /cvs/src/share/man/man4/wg.4,v
retrieving revision 1.7
diff -u -p -r1.7 wg.4
--- man4/wg.4   27 Nov 2020 14:04:49 -0000      1.7
+++ man4/wg.4   27 Nov 2020 14:08:56 -0000
@@ -124,7 +124,10 @@ will accept any random 32-byte base64 st
 When an interface has a private key set with
 .Nm wgkey ,
 the corresponding
-public key is shown in the status output of the interface, like so:
+public key is shown in the status output of the interface
+displayed by
+.Xr ifconfig 8
+when run as root, like so:
 .Bd -literal -offset indent
 wgpubkey NW5l2q2MArV5ZXpVXSZwBOyqhohOf8ImDgUB+jPtJps=
 .Ed


> i agree a bit of explanation makes sense here. saying that, you should
> really join "create interfaces" and "set random private keys" with a
> semicolon or "and" or somesuch, not a comma.

I've committed that with a semicolon.

Reply via email to