On 2020/01/14 21:48, Stuart Henderson wrote:
> On 2020/01/14 21:03, Tobias Heider wrote:
> > On Tue, Jan 14, 2020 at 09:17:11AM -0700, Theo de Raadt wrote:
> > > Stuart Henderson <s...@spacehopper.org> wrote:
> > > 
> > > > On 2020/01/13 20:51, Klemens Nanni wrote:
> > > > > I'm in favour of removing the option and OK with your diff, but simply
> > > > > removing it is probably a bad idea given its nature.
> > > > > 
> > > > > What about printing a deprecation warning so that users can safely
> > > > > adjust their rcctl flags instead of running into "iked(failed)" on the
> > > > > next snapshot.
> > > > 
> > > > Yes please make -6 a noop or a warning rather than an error. Sometimes
> > > > breakage is unavoidable, but this isn't one of those cases.
> > > 
> > > I agree.
> > > 
> > 
> > Makes sense. I added a warning and a notice in current.html.
> > 
> > ok?
> > 
> > Index: www/faq/current.html
> > ===================================================================
> > RCS file: /cvs/www/faq/current.html,v
> > retrieving revision 1.1017
> > diff -u -p -r1.1017 current.html
> > --- www/faq/current.html    31 Dec 2019 02:18:01 -0000      1.1017
> > +++ www/faq/current.html    14 Jan 2020 19:32:25 -0000
> > @@ -135,6 +135,12 @@ or they can be rebuilt from ports.
> >  </b><!--
> >  --></pre>
> >  
> > +<h3 id="r20200114">2020/1/14 - iked(8) automatic IPv6 blocking removed 
> > </h3>
> > +
> > +<a href="https://man.openbsd.org/iked.8";>iked(8)</a> no longer 
> > automatically adds
> > +an IPv6 blocking IPsec flow.
> > +The <code>-6</code> option is deprecated and should be removed from
> > +<a 
> > href="https://man.openbsd.org/rc.conf.local.8";><code>/etc/rc.conf.local</code></a>.
> 
> How about this?
> 
> 
> Index: current.html
> ===================================================================
> RCS file: /cvs/www/faq/current.html,v
> retrieving revision 1.1017
> diff -u -p -r1.1017 current.html
> --- current.html      31 Dec 2019 02:18:01 -0000      1.1017
> +++ current.html      14 Jan 2020 21:47:35 -0000
> @@ -136,6 +136,33 @@ or they can be rebuilt from ports.
>  --></pre>
>  
>  
> +<h3 id="r20200114">2020/1/14 - iked(8) automatic IPv6 blocking removed </h3>
> +
> +<a href="https://man.openbsd.org/iked.8";>iked(8)</a> no longer automatically
> +blocks unencrypted outbound IPv6 packets.
> +This feature was intended to avoid accidental leakage, but in practice was
> +found to mostly be a cause of misconfiguration.
> +The <code>-6</code> flag was used to disable this feature but is now no 
> longer
> +needed and should be removed from <code><a
> +href="https://man.openbsd.org/rc.conf.local.8";>/etc/rc.conf.local</a></code>
> +if used.<p>
> +
> +Instead, if you would like to explicitly block these packets, add the 
> following

Actually, on reading it back now I've posted it, "instead" is bad here,
with the previous sentence it makes it seem like this is something to do
if you *did* use -6, when actually it's something to do if you *didn't*
use -6 and want to keep the feature.

...

So here's some reordering that works better:

Index: current.html
===================================================================
RCS file: /cvs/www/faq/current.html,v
retrieving revision 1.1017
diff -u -p -r1.1017 current.html
--- current.html        31 Dec 2019 02:18:01 -0000      1.1017
+++ current.html        14 Jan 2020 21:53:31 -0000
@@ -136,6 +136,34 @@ or they can be rebuilt from ports.
 --></pre>
 
 
+<h3 id="r20200114">2020/1/14 - iked(8) automatic IPv6 blocking removed </h3>
+
+<a href="https://man.openbsd.org/iked.8";>iked(8)</a> no longer automatically
+blocks unencrypted outbound IPv6 packets.
+This feature was intended to avoid accidental leakage, but in practice was
+found to mostly be a cause of misconfiguration.
+Instead, if you would like to explicitly block these packets, add the following
+line to <code><a 
href="https://man.openbsd.org/ipsec.conf.5";>/etc/ipsec.conf</a></code>
+(<b>not</b> iked.conf):
+
+<pre class="cmdbox">
+<b>flow esp out from ::/0 to ::/0 type deny</b>
+</pre>
+
+and enable loading it with
+
+<pre class="cmdbox">
+# <b>rcctl enable ipsec</b>           # to load at boot
+# <b>ipsecctl -f /etc/ipsec.conf</b>  # to load immediately
+</pre>
+
+If you previously used <a href="https://man.openbsd.org/iked.8";>iked(8)</a>'s
+<code>-6</code> flag to disable this feature, it is no longer needed and should
+be removed from <code><a
+href="https://man.openbsd.org/rc.conf.local.8";>/etc/rc.conf.local</a></code>
+if used.<p>
+
+
 <!--
      Two blank lines before new sections.
      New sentences start on new lines.

Reply via email to