Alexander Klimov <grandmas...@al2klimov.de> wrote:

> First of all, my compliment.
> The installer is already quite ergonomic (for a CLI ;) ).
> But there are the following two little diff(1)s standing
> between it and its perfection IMAO.
> 
> 
> --- distrib/miniroot/install.sub.orig   Thu May 18 12:37:52 2023
> +++ distrib/miniroot/install.sub        Thu May 18 12:44:49 2023
> @@ -1220,3 +1220,3 @@
>                 ask_until "IPv6 address for $_if? (or 'autoconf' or 'none')" \
> -                         "${_addr:-none}"
> +                         "${_addr:-autoconf}"
>                 case $resp in
> 
> I personally enable IPv6 everywhere,
> even if I have only link-local addresses.
> If I got SLAAC, nice for my OpenBSD clients
> and the clients of my OpenBSD servers.
> Win-win. If not, I haven't lost anything.
> In the worst case I have to do specific config,
> but then the default doesn't matter anyway.
> 
> The only reason against this could be a permit-default pf.conf.
> But such shouldn't be done and this is the installer after all.
> One writes pf.conf after the installer or can -in extreme case-
> still type "none" here (which is shorter to type).
> I know that you folks like not to surprise users.
> But IMAO default-enabling IPv6 *on new installs* isn't a surprise
> (in 2023 when IIRC some US gov orgs already sell their whole IPv4s).

I disagree.  I believe network participation should be opt-in.  Minimal
network configuration is encouraged, but the minimum is v4, not v6.

v6 is still not natural.  It is a surprise. I think it will be too easy
for users to  turn it on in the wrong places, and since OpenBSD
automatically learns DNS configuration from such v6 network configuration
edges, this is a hazard.  Configurating v6 must be intentional.

> In case you don't agree with me:
> What about a shortcut "a" (= autoconf)
> for IPv[46] address (like below)?

That would make sense.  It would also need to be done for v4.  Diff would
be something like this, not tested, there might be other pieces missing to
do it perfectly.

Index: install.sub
===================================================================
RCS file: /cvs/src/distrib/miniroot/install.sub,v
retrieving revision 1.1244
diff -u -p -u -r1.1244 install.sub
--- install.sub 2 May 2023 15:55:58 -0000       1.1244
+++ install.sub 18 May 2023 14:01:01 -0000
@@ -1105,7 +1105,7 @@ v4_config() {
                case $resp in
                none)   return
                        ;;
-               autoconf|dhcp)
+               a|autoconf|dhcp)
                        dhcp_request $_if
                        echo "inet autoconf" >>$_hn
                        return
@@ -1222,7 +1222,7 @@ v6_config() {
                case $resp in
                none)   return
                        ;;
-               autoconf)
+               a|autoconf)
                        ifconfig $_if inet6 autoconf up
                        echo "inet6 autoconf" >>$_hn
                        return

Reply via email to