[prev in list] [next in list] [prev in thread] [next in thread] 

List:       libreswan-dev
Subject:    Re: [Swan-dev] [Swan-commit] Changes to ref refs/heads/main
From:       Andrew Cagney via Swan-dev <swan-dev () lists ! libreswan ! org>
Date:       2024-04-21 2:56:08
Message-ID: CAJeAr6voR8GZsrH1N7L=5gwe8=VoYn1G-0CU2=Oaynmvwc_jTw () mail ! gmail ! com
[Download RAW message or body]

On Sat, 20 Apr 2024 at 19:40, Paul Wouters via Swan-dev
<swan-dev@lists.libreswan.org> wrote:
>
> On Sat, 20 Apr 2024, Andrew Cagney via Swan-commit wrote:
>
> >    libipsecconf: rename internal enum AUTOSTART_ONDEMAND -> AUTOSTART_ROUTE
>
> This is wrong. The libipsecconf names match the _keywords_ used by auto=
> and auto=route has been long obsoleted for auto=ondemand.

And auto=ondemand makes no sense when the connection is never-negotiate.

I'll define AUTOSTART_ROUTE, AUTOSTART_ONDEMAND, AUTOSTART_START,
AUTOSTART_ADD so that pluto can see exactly what the config file
contained.

> >    consistent with other code
>
> Internal code does not matter much. It is the mapping user option to
> variable name that should be consistent in code.

It does matter.  The closer the alignment between the UI and the
internals the easier it is to describe and understand a behaviour.
And here all the internals consistently use route/unroute.  For
instance, routing an RT_UNROUTED connection transitions it to either
RT_ROUTED_ONDEMAND or RT_ROUTED_NEVER_NEGOTIATE.
_______________________________________________
Swan-dev mailing list
Swan-dev@lists.libreswan.org
https://lists.libreswan.org/mailman/listinfo/swan-dev
[prev in list] [next in list] [prev in thread] [next in thread]