NetBSD Problem Report #58015

From www@netbsd.org  Sat Mar  9 12:08:42 2024
Return-Path: <www@netbsd.org>
Received: from mail.netbsd.org (mail.netbsd.org [199.233.217.200])
	(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits))
	(Client CN "mail.NetBSD.org", Issuer "mail.NetBSD.org CA" (not verified))
	by mollari.NetBSD.org (Postfix) with ESMTPS id 606441A923C
	for <gnats-bugs@gnats.NetBSD.org>; Sat,  9 Mar 2024 12:08:42 +0000 (UTC)
Message-Id: <20240309120841.063411A923F@mollari.NetBSD.org>
Date: Sat,  9 Mar 2024 12:08:40 +0000 (UTC)
From: campbell+netbsd@mumble.net
Reply-To: campbell+netbsd@mumble.net
To: gnats-bugs@NetBSD.org
Subject: wg(4) man page issues: missing umask, no IPv6, wrong argument
X-Send-Pr-Version: www-1.0

>Number:         58015
>Category:       misc
>Synopsis:       wg(4) man page issues: missing umask, no IPv6, wrong argument
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    riastradh
>State:          closed
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Sat Mar 09 12:10:00 +0000 2024
>Closed-Date:    Fri Apr 19 02:33:05 +0000 2024
>Last-Modified:  Fri Apr 19 02:33:05 +0000 2024
>Originator:     Taylor R Campbell
>Release:        current, 10
>Organization:
The NotWG Foundation
>Environment:
>Description:
1. wg(4) example setup leaves /etc/wg/wg0 private key world-readable
2. wg(4) example is IPv4-only
3. wgconfig(8) mentions --preshared-key-file=... but the option is --preshared-key=...
>How-To-Repeat:
man wg
man wgconfig
>Fix:
Yes, please!

>Release-Note:

>Audit-Trail:
From: "Taylor R Campbell" <riastradh@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/58015 CVS commit: src/share/man/man4
Date: Sat, 9 Mar 2024 12:36:07 +0000

 Module Name:	src
 Committed By:	riastradh
 Date:		Sat Mar  9 12:36:07 UTC 2024

 Modified Files:
 	src/share/man/man4: wg.4

 Log Message:
 wg(4): Spruce up example a bit.

 - Suggest umask so the private keys aren't world readable.
 - Suggest use of pre-shared key files.
 - Use TEST-NET-1 and TEST-NET-2 addresses for the example instead of
   real publicly routable addresses.

 Holding off on adding IPv6 example until the tun(4) issue is fixed
 (PR bin/58013).

 PR misc/58015


 To generate a diff of this commit:
 cvs rdiff -u -r1.6 -r1.7 src/share/man/man4/wg.4

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

State-Changed-From-To: open->needs-pullups
State-Changed-By: riastradh@NetBSD.org
State-Changed-When: Sun, 10 Mar 2024 15:26:10 +0000
State-Changed-Why:
let's get the commit so far pulled up, deal with IPv6 later


State-Changed-From-To: needs-pullups->pending-pullups
State-Changed-By: riastradh@NetBSD.org
State-Changed-When: Mon, 11 Mar 2024 01:14:20 +0000
State-Changed-Why:
pullup-10 #629
inapplicable <10 (no wg)


From: "Martin Husemann" <martin@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/58015 CVS commit: [netbsd-10] src/share/man/man4
Date: Mon, 11 Mar 2024 19:39:23 +0000

 Module Name:	src
 Committed By:	martin
 Date:		Mon Mar 11 19:39:23 UTC 2024

 Modified Files:
 	src/share/man/man4 [netbsd-10]: wg.4

 Log Message:
 Pull up following revision(s) (requested by riastradh in ticket #629):

 	share/man/man4/wg.4: revision 1.7

 wg(4): Spruce up example a bit.
 - Suggest umask so the private keys aren't world readable.
 - Suggest use of pre-shared key files.
 - Use TEST-NET-1 and TEST-NET-2 addresses for the example instead of
   real publicly routable addresses.

 Holding off on adding IPv6 example until the tun(4) issue is fixed
 (PR bin/58013).

 PR misc/58015


 To generate a diff of this commit:
 cvs rdiff -u -r1.6 -r1.6.6.1 src/share/man/man4/wg.4

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

State-Changed-From-To: pending-pullups->open
State-Changed-By: riastradh@NetBSD.org
State-Changed-When: Mon, 11 Mar 2024 22:27:36 +0000
State-Changed-Why:
- umask issue fixed
- IPv6 example still to be written
- wrong argument already addressed by PR 57663 fix


State-Changed-From-To: open->needs-pullups
State-Changed-By: riastradh@NetBSD.org
State-Changed-When: Tue, 12 Mar 2024 02:29:02 +0000
State-Changed-Why:
added IPv6 to example, needs pullup-10


From: "Taylor R Campbell" <riastradh@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/58015 CVS commit: src/share/man/man4
Date: Tue, 12 Mar 2024 02:28:03 +0000

 Module Name:	src
 Committed By:	riastradh
 Date:		Tue Mar 12 02:28:03 UTC 2024

 Modified Files:
 	src/share/man/man4: wg.4

 Log Message:
 wg(4): Rework example numbering for clarity and add IPv6.

 Let's avoid triggering unease with host number 0.

 PR misc/58015


 To generate a diff of this commit:
 cvs rdiff -u -r1.7 -r1.8 src/share/man/man4/wg.4

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

From: "Taylor R Campbell" <riastradh@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/58015 CVS commit: src/share/man/man4
Date: Sat, 23 Mar 2024 21:34:07 +0000

 Module Name:	src
 Committed By:	riastradh
 Date:		Sat Mar 23 21:34:07 UTC 2024

 Modified Files:
 	src/share/man/man4: wg.4

 Log Message:
 wg(4): Fix IPv6 numbering in example diagram.

 This way it matches the configuration suggested below (which avoids
 host number zero on the subnet).

 PR misc/58015


 To generate a diff of this commit:
 cvs rdiff -u -r1.8 -r1.9 src/share/man/man4/wg.4

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

Responsible-Changed-From-To: misc-bug-people->riastradh
Responsible-Changed-By: riastradh@NetBSD.org
Responsible-Changed-When: Thu, 04 Apr 2024 19:25:57 +0000
Responsible-Changed-Why:
pullup-10 #658
inapplicable <10


State-Changed-From-To: needs-pullups->pending-pullups
State-Changed-By: riastradh@NetBSD.org
State-Changed-When: Thu, 04 Apr 2024 19:25:57 +0000
State-Changed-Why:
mine


From: "Martin Husemann" <martin@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/58015 CVS commit: [netbsd-10] src/share/man/man4
Date: Thu, 18 Apr 2024 15:44:38 +0000

 Module Name:	src
 Committed By:	martin
 Date:		Thu Apr 18 15:44:37 UTC 2024

 Modified Files:
 	src/share/man/man4 [netbsd-10]: wg.4

 Log Message:
 Pull up following revision(s) (requested by riastradh in ticket #658):

 	share/man/man4/wg.4: revision 1.8
 	share/man/man4/wg.4: revision 1.9

 wg(4): Rework example numbering for clarity and add IPv6.

 Let's avoid triggering unease with host number 0.
 PR misc/58015

 wg(4): Fix IPv6 numbering in example diagram.

 This way it matches the configuration suggested below (which avoids
 host number zero on the subnet).

 PR misc/58015


 To generate a diff of this commit:
 cvs rdiff -u -r1.6.6.1 -r1.6.6.2 src/share/man/man4/wg.4

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

State-Changed-From-To: pending-pullups->closed
State-Changed-By: riastradh@NetBSD.org
State-Changed-When: Fri, 19 Apr 2024 02:33:05 +0000
State-Changed-Why:
fixed and pulled up


>Unformatted:

Home	PR Database Search