On Sat, Aug 20, 2022 at 02:25:25PM +0100, Stuart Henderson wrote: > > > > Long output with <if> as an argument, wgpeers section present: > > > > pce-0067# ifconfig.ifaliases wg0 > > wg0: flags=80c3<UP,BROADCAST,RUNNING,NOARP,MULTICAST> mtu 1420 > > index 8 priority 0 llprio 3 > > wgport 51820 > > wgpubkey qcb... > > wgpeer klM... > > wgpsk (present) > > wgpka 25 (sec) > > wgendpoint xxx.xxx.xxx.xxx 51820 > > tx: 178764, rx: 65100 > > last handshake: 7 seconds ago > > wgaip fde4:f456:48c2:13c0::/64 > > groups: wg > > inet6 fde4:f456:48c2:13c0::cc67 prefixlen 64 > > > > > > Above long output works with group as an argument (ifconfig wg) and with > > option -A (ifconfig -A), so I think from user experience perspective, > > works as expected. > > > > Manual page changes not provided, as I'm not sure are they needed with > > this diff. > > At least a small change is needed. Maybe some different text would > be more appropriate though. > > Index: ifconfig.8 > =================================================================== > RCS file: /cvs/src/sbin/ifconfig/ifconfig.8,v > retrieving revision 1.384 > diff -u -p -r1.384 ifconfig.8 > --- ifconfig.8 27 Jun 2022 16:27:03 -0000 1.384 > +++ ifconfig.8 20 Aug 2022 13:22:43 -0000 > @@ -75,7 +75,7 @@ Only the superuser may modify the config > The following options are available: > .Bl -tag -width Ds > .It Fl A > -Causes full interface alias information for each interface to > +Causes full interface alias and wgpeer information for each interface to > be displayed. > .It Fl a > Causes >
hi. i think this just muddies the simple explanation of -A. i don;t think it's worth it for a single exception. what about amending wg(1) to note that this info is available via -A? then wg users can find it. jmc